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

Added the ability to continue without a token #51

Open
wants to merge 3 commits into
base: master
Choose a base branch
from
Open

Added the ability to continue without a token #51

wants to merge 3 commits into from

Conversation

KieronWiltshire
Copy link

Here at Concept Gaming, we're implementing GraphQL. The setup makes things difficult to work with as GraphQL only exposes data through the use of a single route, this requires us to explicitly check specific end points. I'd greatly appreciate it if this could be merged rather than having us to re-implement it in our application.

This allows the ability to explicitly check specific routes.
@markstos
Copy link

@KieronWiltshire Have you confirmed your change is compliant with the OAuth 2 RFC? Please reference the sections of the RFC that apply to your change.

@KieronWiltshire
Copy link
Author

@markstos it complies with the entire spec, it just allows you to catch the error and continue.

@markstos
Copy link

@KieronWiltshire It would be helpful if you linked to the specific spec section that addresses continuing without a token.

@KieronWiltshire
Copy link
Author

@markstos the spec doesn't specify, to be completely honest, this isn't something I need anymore, well not as of right now, so feel free to close this merge request.

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