Fix JWT Token Refresh (#209)

Author: Jeff-Meadows

boxsdk/auth/jwt_auth.py

@@ -291,5 +291,7 @@ def _refresh(self, access_token):
         """
         # pylint:disable=unused-argument
         if self._user_id is None:
-            return self.authenticate_instance()
-        return self.authenticate_user()
+            new_access_token = self.authenticate_instance()
+        else:
+            new_access_token = self.authenticate_user()
+        return new_access_token, None

boxsdk/auth/oauth2.py

@@ -203,12 +203,11 @@ def refresh(self, access_token_to_refresh):
             # The lock here is for handling that case that multiple requests fail, due to access token expired, at the
             # same time to avoid multiple session renewals.
             if (access_token is None) or (access_token_to_refresh == access_token):
-                # If the active access token is the same as the token needs to
+                # If the active access token is the same as the token that needs to
                 # be refreshed, or if we don't currently have any active access
                 # token, we make the request to refresh the token.
-                return self._refresh(access_token_to_refresh)
-
-            # If the active access token (self._access_token) is not the same as the token needs to be refreshed,
+                access_token, refresh_token = self._refresh(access_token_to_refresh)
+            # Else, if the active access token (self._access_token) is not the same as the token needs to be refreshed,
             # it means the expired token has already been refreshed. Simply return the current active tokens.
             return access_token, refresh_token