Wrap scalar values in tuples instead of lists

Author: bjmc

jose/jws.py

@@ -222,8 +222,8 @@ def _get_keys(key):
     elif (isinstance(key, Iterable) and
           not (isinstance(key, six.string_types) or isinstance(key, Mapping))):
         return key
-    else:  # Scalar value, wrap in list.
-        return [key]
+    else:  # Scalar value, wrap in tuple.
+        return (key,)
 
 
 def _verify_signature(signing_input, header, signature, key='', algorithms=None):

jose/jwt.py

@@ -351,7 +351,7 @@ def _validate_iss(claims, issuer=None):
 
     if issuer is not None:
         if isinstance(issuer, string_types):
-            issuer = [issuer]
+            issuer = (issuer,)
         if claims.get('iss') not in issuer:
             raise JWTClaimsError('Invalid issuer')
 

tests/test_jws.py

@@ -201,13 +201,13 @@ def jwk_set():
 class TestGetKeys(object):
 
     def test_dict(self):
-        assert [{}] == jws._get_keys({})
+        assert ({},) == jws._get_keys({})
 
     def test_custom_object(self):
         class MyDict(dict):
             pass
         mydict = MyDict()
-        assert [mydict] == jws._get_keys(mydict)
+        assert (mydict,) == jws._get_keys(mydict)
 
     def test_RFC7517_string(self):
         key = '{"keys": [{}, {}]}'
@@ -218,7 +218,7 @@ def test_RFC7517_mapping(self):
         assert [{}, {}] == jws._get_keys(key)
 
     def test_string(self):
-        assert ['test'] == jws._get_keys('test')
+        assert ('test',) == jws._get_keys('test')
 
     def test_tuple(self):
         assert ('test', 'key') == jws._get_keys(('test', 'key'))