Fix: ConfigParser incorrectly parses tokens with @ character

Author: remip2

stormshield/sns/configparser.py

@@ -94,7 +94,7 @@ def __init__(self, text):
                     line = line.encode('utf-8')
                 # parse token=value token2=value2
                 lexer = shlex(line, posix=True)
-                lexer.wordchars += "=.-*:,/"
+                lexer.wordchars += "=.-*:,/@"
                 parsed = {}
                 for word in lexer:
                     # ignore anything else than token=value

stormshield/sns/sslclient/__version__.py

@@ -2,4 +2,4 @@
 # major: breaking API change
 # minor: new functionality
 # patch: bugfix
-__version__ = "1.0.2"
+__version__ = "1.0.3"

tests/test_configparser.py

@@ -173,7 +173,7 @@ def test_empty_value(self):
         self.assertEqual(expected, config.data)
 
     def test_slash_in_value(self):
-        """ test parsing of value with slash character """
+        """ Test parsing of value with slash character """
 
         input = """101 code=00a01000 msg="Begin" format="section_line"
 [StaticRoutes]
@@ -187,6 +187,21 @@ def test_slash_in_value(self):
         config = ConfigParser(input)
         self.assertEqual(expected, config.data)
 
+    def test_arobase_in_value(self):
+        """ Test parsing of value with @ character """
+
+        input = """101 code=00a01000 msg="Begin" format="section_line"
+[Result]
+ruleid=1 state=on action=pass from=*@* to=*@* comment="default rule (pass all)"
+100 code=00a00100 msg="Ok"\""""
+
+        expected = {'Result': [
+            {'ruleid': '1', 'state': 'on', 'action': 'pass', 'from': '*@*',
+             'to': '*@*', 'comment': 'default rule (pass all)'}]}
+
+        config = ConfigParser(input)
+        self.assertEqual(expected, config.data)
+
 
 if __name__ == '__main__':
     unittest.main()