[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