Skip to content
This repository was archived by the owner on May 1, 2024. It is now read-only.

Add new property to disable the escaping of scope parameter. #62

Open
wants to merge 1 commit into
base: master
Choose a base branch
from

Conversation

jsauve
Copy link

@jsauve jsauve commented Apr 28, 2014

By default, the OAuth2Authenticator escapes the scope parameter. When multiple scopes are provided (example: "basic+comments+relationships+likes"), some OAuth2 providers (such as Instagram) return an HTTP 400 BAD REQUEST on authentication. This is because the encoding of the "+" character to "%2B" is not accepted by some OAuth2 providers (such as Instagram). This commit adds a property named DoNotEscapeScope that can be set to true to circumvent the issue.

… multiple scopes are provided (example: "basic+comments+relationships+likes"), some OAuth2 providers (such as Instagram) return an HTTP 400 BAD REQUEST on authentication. This is because the encoding of the "+" character is not accepted by some OAuth2 providers (such as Instagram). This commit adds a property named DoNotEscapeScope that can be set to true to circumvent the issue.
Sign up for free to subscribe to this conversation on GitHub. Already have an account? Sign in.
Labels
None yet
Projects
None yet
Development

Successfully merging this pull request may close these issues.

1 participant