Skip to content
New issue

Have a question about this project? Sign up for a free GitHub account to open an issue and contact its maintainers and the community.

By clicking “Sign up for GitHub”, you agree to our terms of service and privacy statement. We’ll occasionally send you account related emails.

Already on GitHub? Sign in to your account

Fix "Scopes must be set on post auth" #320

Open
wants to merge 6 commits into
base: master
Choose a base branch
from

Commits on Mar 16, 2017

  1. Fix "Scopes must be set on post auth." exception

    According to the OAuth RFC, the Authorization Code grant can be requested without explicitly providing a list of scopes. This change enables that behavior by using the default scopes the OAuth request was initially populated with.
    ibrewster committed Mar 16, 2017
    Configuration menu
    Copy the full SHA
    ad89345 View commit details
    Browse the repository at this point in the history

Commits on Mar 17, 2017

  1. Merge pull request #1 from ibrewster/ibrewster-patch-1

    Fix "Scopes must be set on post auth." exception
    ibrewster committed Mar 17, 2017
    Configuration menu
    Copy the full SHA
    1dfc86f View commit details
    Browse the repository at this point in the history
  2. Use locally defined scopes variable

    Use the locally defined scopes variable rather than trying to pull from the request object, which may not exist in certain requests.
    ibrewster committed Mar 17, 2017
    Configuration menu
    Copy the full SHA
    69a121c View commit details
    Browse the repository at this point in the history
  3. Check to see where we can find scopes

    Check to make sure scopes is defined locally or request exists in kwargs and has a scopes attribute. If not, fall back to the old behavior of not declaring default scopes and hope they are not needed.
    ibrewster committed Mar 17, 2017
    Configuration menu
    Copy the full SHA
    c90bd4f View commit details
    Browse the repository at this point in the history

Commits on Mar 24, 2017

  1. Remove unneeded line

    Remove unused line that wasn't being covered by tests.
    ibrewster committed Mar 24, 2017
    Configuration menu
    Copy the full SHA
    bce6406 View commit details
    Browse the repository at this point in the history
  2. Configuration menu
    Copy the full SHA
    972122b View commit details
    Browse the repository at this point in the history