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.

Sign up for GitHub

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

Jump to bottom

Bugfix: Allow both Read and Write if both are present in the scope #1

Open

jpoissant wants to merge 1 commit into tom-smith-okta:master from jpoissant:allow_read_and_write

jpoissant commented Feb 13, 2020

Bug: if you provide both read and write scope api:read api:write only the GET method would be allowed.

Fix: Change the else if to an if when checking for api:write in the scp


          Bugfix: Allow both Read and Write if both are present in the scope

15674e1

jpoissant requested a review from tom-smith-okta

February 13, 2020 18:22

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

Labels

None yet