Skip to content
This repository has been archived by the owner on Mar 8, 2022. It is now read-only.

Fix null scope issue #162

Merged
merged 1 commit into from
Jan 16, 2020
Merged

Fix null scope issue #162

merged 1 commit into from
Jan 16, 2020

Conversation

alexkappa
Copy link
Owner

Community Note

  • Please vote on this pull request by adding a 👍 reaction to the original pull request comment to help the community and maintainers prioritize this request
  • Please do not leave "+1" comments, they generate extra noise for pull request followers and do not help prioritize the request

Fixes #159

Changes proposed in this pull request:

  • Check if scope was defined. If not, create an empty slice instead of leaving null.
  • Add test to verify:
    • Creating a auth0_client_grant with no scopes (scope = [ ]).
    • Updating to add a scope (scope = [ "create:foo" ]).
    • Updating again to remove any scopes (scope = [ ]).

Sign up for free to subscribe to this conversation on GitHub. Already have an account? Sign in.
Projects
None yet
Development

Successfully merging this pull request may close these issues.

auth0_client_grant does not accept empty scopes
1 participant