Merge pull request #416 from splunk/DVPL-10302

Removed strip() method in load_value() method from data.py file

Author: akaila-splunk

splunklib/data.py

@@ -161,8 +161,8 @@ def load_value(element, nametable=None):
         text = element.text
         if text is None: 
             return None
-        text = text.strip()
-        if len(text) == 0: 
+
+        if len(text.strip()) == 0:
             return None
         return text
 

tests/test_storage_passwords.py

@@ -222,6 +222,16 @@ def test_delete(self):
         self.storage_passwords.delete(username + "/foo", "/myrealm")
         self.assertEqual(start_count, len(self.storage_passwords))
 
+    def test_spaces_in_username(self):
+        start_count = len(self.storage_passwords)
+        realm = testlib.tmpname()
+        username = "  user1  "
+
+        p = self.storage_passwords.create("changeme", username, realm)
+        self.assertEqual(p.username, username)
+
+        p.delete()
+        self.assertEqual(start_count, len(self.storage_passwords))
 
 if __name__ == "__main__":
     try: