[tor-commits] [nyx/master] color_override config broken if different case
atagar at torproject.org
atagar at torproject.org
Tue Oct 27 01:15:52 UTC 2020
commit ed031ca750be06e4134d58b81448e60020c0cfd8
Author: Damian Johnson <atagar at torproject.org>
Date: Mon Oct 26 18:06:18 2020 -0700
color_override config broken if different case
Oops, our code that validates color values normalizes into camel case, but
doesn't persist that normalization for its usage. As a result Nyx gets into a
pretty broken state. Caught thanks to Sai.
https://github.com/torproject/nyx/issues/30
---
nyx/curses.py | 2 ++
1 file changed, 2 insertions(+)
diff --git a/nyx/curses.py b/nyx/curses.py
index 98b1d11..3c3fa69 100644
--- a/nyx/curses.py
+++ b/nyx/curses.py
@@ -173,6 +173,8 @@ def conf_handler(key, value):
if value not in Color and value != 'None':
raise ValueError('"%s" isn\'t a valid color' % value)
+
+ return value
elif key == 'max_line_wrap':
return max(1, value)
More information about the tor-commits
mailing list