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

Request for granting access token can contain JSON body. #63

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

Conversation

robinvdvleuten
Copy link

Hi there. I modified the grantAccessToken method a little bit to accept both post and json bodies.

@Spomky
Copy link

Spomky commented Jul 13, 2014

This behaviour is not described in the RFC nor in the drafts.

@robinvdvleuten
Copy link
Author

Yeah I know it's not described in the official docs, but some APIs out there provide the functionalilty. And it does make sense if everything in your API is json, to have the oauth endpoints support json as well.

@MobileD-pro
Copy link

Can you please merge this?

Even though this behaviour is not described in the RFC, all APIs accept this type of request
https://github.com/api-platform/api-platform uses your repository and accepts it.

Should we change our front code only to OAuth ? this is restrictive.

#111 duplicate this.

Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Projects
None yet
Development

Successfully merging this pull request may close these issues.

4 participants