This repository has been archived by the owner on Oct 29, 2024. It is now read-only.
Refactor Scope Handling to String Type in Compliance with OAuth RFC Standards #15
Add this suggestion to a batch that can be applied as a single commit.
This suggestion is invalid because no changes were made to the code.
Suggestions cannot be applied while the pull request is closed.
Suggestions cannot be applied while viewing a subset of changes.
Only one suggestion per line can be applied in a batch.
Add this suggestion to a batch that can be applied as a single commit.
Applying suggestions on deleted lines is not supported.
You must change the existing code in this line in order to create a valid suggestion.
Outdated suggestions cannot be applied.
This suggestion has been applied or marked resolved.
Suggestions cannot be applied from pending reviews.
Suggestions cannot be applied on multi-line comments.
Suggestions cannot be applied while the pull request is queued to merge.
Suggestion cannot be applied right now. Please check back later.
This pull request introduces a significant update to OAuth implementation, specifically focusing on the handling of the
scopes
property. In alignment with OAuth RFC standards, we have transitioned thescopes
property from being an array to a single string. This change ensures compliance with OAuth specifications and enhances the interoperability of OAuth handling.Changes
scopes
property across the application, previously an array of strings, has been updated to a single space-delimited string.scopes
are processed, parsed, or stored to reflect this change.scopes
property. Tests now validate against a single string rather than an array of strings, ensuring consistency with OAuth RFC recommendations.Impact
Background
According to the OAuth RFC, the
scope
parameter should be represented as a list of space-delimited, case-sensitive strings. This format allows for a more flexible and standardized way of representing scopes, facilitating better interoperability across different systems and OAuth providers.