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

Skips authorization form if the client has been authorized by the resource owner #1

Merged
merged 1 commit into from
Nov 30, 2011

Conversation

felipeelias
Copy link
Member

Even though the OAuth 2 spec does not mention this behavior, it is a common practice to skip the authorization form and redirect the resource owner straight to the client when the client was previously authorized.

This commit also includes a small refactor in the authorization request spec, moving common parts to its proper helpers

…esource owner

Even though the OAuth 2 spec does not mention this behavior, it is
a common practice to skip the authorization form and redirect the
resource owner straight to the client when the client was previously
authorized.

This commit also includes a small refactor in the authorization
request spec, moving common parts to its proper helpers
@piotrj
Copy link
Member

piotrj commented Nov 30, 2011

Looks good.

piotrj added a commit that referenced this pull request Nov 30, 2011
Skips authorization form if the client has been authorized by the resource owner
@piotrj piotrj merged commit dab436e into doorkeeper-gem:master Nov 30, 2011
tute pushed a commit that referenced this pull request Sep 7, 2014
Reverts test-related changes
Paradoxor referenced this pull request in Paradoxor/doorkeeper Mar 30, 2020
danielcooper added a commit to danielcooper/doorkeeper that referenced this pull request Mar 15, 2022
danielcooper added a commit to danielcooper/doorkeeper that referenced this pull request Mar 18, 2022
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
None yet
Projects
None yet
Development

Successfully merging this pull request may close these issues.

2 participants