Remove unnecessary Unicode decoding before json.loads() (jpadilla#542)

Since Python 3.6, json.loads() accepts both Unicode and byte strings.

https://docs.python.org/3/library/json.html#json.loads

> Changed in version 3.6: s can now be of type bytes or bytearray. The
> input encoding should be UTF-8, UTF-16 or UTF-32.

Author: jdufresne

jwt/api_jws.py

@@ -188,7 +188,7 @@ def _load(self, jwt):
             raise DecodeError("Invalid header padding") from err
 
         try:
-            header = json.loads(header_data.decode("utf-8"))
+            header = json.loads(header_data)
         except ValueError as e:
             raise DecodeError("Invalid header string: %s" % e) from e
 

jwt/api_jwt.py

@@ -95,9 +95,9 @@ def decode(
 
         try:
             if complete:
-                payload = json.loads(decoded["payload"].decode("utf-8"))
+                payload = json.loads(decoded["payload"])
             else:
-                payload = json.loads(decoded.decode("utf-8"))
+                payload = json.loads(decoded)
         except ValueError as e:
             raise DecodeError("Invalid payload string: %s" % e)
         if not isinstance(payload, dict):

tests/test_api_jws.py

@@ -232,7 +232,7 @@ def test_decodes_valid_es384_jws(self, jws):
         decoded_payload = jws.decode(
             example_jws, example_pubkey, algorithms=["ES256"]
         )
-        json_payload = json.loads(force_unicode(decoded_payload))
+        json_payload = json.loads(decoded_payload)
 
         assert json_payload == example_payload
 
@@ -262,7 +262,7 @@ def test_decodes_valid_rs384_jws(self, jws):
         decoded_payload = jws.decode(
             example_jws, example_pubkey, algorithms=["RS384"]
         )
-        json_payload = json.loads(force_unicode(decoded_payload))
+        json_payload = json.loads(decoded_payload)
 
         assert json_payload == example_payload
 
@@ -699,7 +699,7 @@ def default(self, o):
         )
 
         header = force_bytes(force_unicode(token).split(".")[0])
-        header = json.loads(force_unicode(base64url_decode(header)))
+        header = json.loads(base64url_decode(header))
 
         assert "some_decimal" in header
         assert header["some_decimal"] == "it worked"