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

webservice: check OAuth scopes #525

Merged
merged 1 commit into from
Sep 28, 2023

Conversation

jrchamp
Copy link
Collaborator

@jrchamp jrchamp commented Sep 28, 2023

Thanks to OAuth, we know which scopes we have been granted and can make sure that we're running the correct methods based on that. No more "try the dashboard endpoint and if there's an exception, try the report endpoint".

@jrchamp jrchamp added the bug Fixes problems or reduces technical debt label Sep 28, 2023
@jrchamp jrchamp requested a review from a team September 28, 2023 06:59
@jrchamp jrchamp self-assigned this Sep 28, 2023
@jrchamp jrchamp merged commit ea77159 into ncstate-delta:main Sep 28, 2023
@jrchamp jrchamp deleted the fix/check-oauth-scopes branch September 28, 2023 16:11
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
bug Fixes problems or reduces technical debt
Projects
No open projects
Archived in project
Development

Successfully merging this pull request may close these issues.

2 participants