Skip to content

Unable to refresh JWT auth token #197

Closed

Description

@GabeLambda

https://github.com/box/box-python-sdk/blob/master/boxsdk/auth/jwt_auth.py#L195

Callers of this method are expecting a tuple of (access_token, refresh_token), not just access_token. This breaks downstream in https://github.com/box/box-python-sdk/blob/master/boxsdk/session/box_session.py#L196, causing clients which use JWTAuth to break on refresh.

Example:

File "/usr/local/lib/python3.4/dist-packages/boxsdk/session/box_session.py", line 196, in _renew_session
    new_access_token, _ = self._oauth.refresh(access_token_used)

ValueError: too many values to unpack (expected 2)

Version: 2.0.0a4

Activity

  1. maxdebayser commented on Jul 4, 2017

    @maxdebayser

    This fix worked for me

  2. cschloer commented on Jul 19, 2017

    @cschloer

    Fixed also worked for me but would rather not use a fork forever. Any status on this getting into master?

  3. jmoldow commented on Jul 26, 2017

    @jmoldow
    Contributor

    Fixed via #209

  4. added 2 commits that reference this issue on Jul 26, 2017
    a65026d
    f77cb41
  5. jmoldow commented on Jul 27, 2017

    @jmoldow
    Contributor

    Also fixed in 1.5 via #217.

    I just released a new version, 1.5.4, with this fix.

Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Metadata

Metadata

Assignees

No one assigned

    Labels

    No labels
    No labels

    Type

    No type

    Projects

    No projects

      Milestone

      No milestone

      Relationships

      None yet

      Development

      No branches or pull requests

      Issue actions