Collapse scopes - we only support one or the other (write includes read)#13887
Collapse scopes - we only support one or the other (write includes read)#13887rickyrombo merged 3 commits intomainfrom
Conversation
|
There was a problem hiding this comment.
Pull request overview
Updates the web OAuth login page’s query parsing to normalize incoming OAuth scope values when multiple scopes are supplied (notably from Swagger UI), so the flow can proceed with a single effective scope.
Changes:
- Added a
collapseScopeshelper to reduce space-separated scope strings (and some array cases) to a single effective scope. - Updated
useParsedQueryParamsto parsescopeasrawScopeand then derivescopeviacollapseScopes.
💡 Add Copilot custom instructions for smarter, more guided reviews. Learn how to get started.
Co-authored-by: Copilot <175728472+Copilot@users.noreply.github.com>
Co-authored-by: Copilot <175728472+Copilot@users.noreply.github.com>
🌐 Web preview readyPreview URL: https://audius-web-preview-pr-13887.audius.workers.dev Unique preview for this PR (deployed from this branch). |
No description provided.