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

#38 - Invalid or expired JWT tokens should prevent request from being serviced #43

Closed

Conversation

jasonbahl
Copy link
Collaborator

  • This validates the Auth Token when the GraphQL Request is being initiated, and if it's invalid the request does not execute and a 403 status is returned.

closes #38

@mlipscombe when you have a chance can you confirm this works with your consuming application without issues?

… serviced

- This validates the Auth Token when the GraphQL Request is being initiated, and if it's invalid the request does not execute and a 403 status is returned.
@jasonbahl jasonbahl added bug status: awaiting author response Additional information has been requested from the author labels Jul 19, 2019
@jasonbahl jasonbahl self-assigned this Jul 19, 2019
@jasonbahl
Copy link
Collaborator Author

@mlipscombe, wanted to check with you to see if this PR resolves your issues. I plan to merge unless I hear otherwise.

@jasonbahl jasonbahl closed this Jan 29, 2020
@jasonbahl jasonbahl deleted the bug/#38-do-not-execute-for-invalid-jwt-tokens branch January 29, 2020 14:48
@henrikwirth
Copy link
Collaborator

@jasonbahl has this been merged?

@jasonbahl
Copy link
Collaborator Author

oh shoot. I think I was doing some branch cleanup and didn't realize it closed this.

@jasonbahl jasonbahl restored the bug/#38-do-not-execute-for-invalid-jwt-tokens branch February 13, 2020 22:41
@jasonbahl jasonbahl reopened this Feb 13, 2020
@jasonbahl
Copy link
Collaborator Author

Already merged to develop

@jasonbahl jasonbahl closed this Feb 17, 2020
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
bug status: awaiting author response Additional information has been requested from the author
Projects
None yet
Development

Successfully merging this pull request may close these issues.

2 participants