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

Nil pointer reference, if refreshtoken is expired or not sent with request #13

Open
iwyg opened this issue Feb 18, 2021 · 2 comments
Open

Comments

@iwyg
Copy link

iwyg commented Feb 18, 2021

Hello there

Although this is a quite unlikely scenario, the auth.Handler will panic, caused by line 412 in auth.go, if the refresh token is not sent with the request.

Maybe error branching after calling err := auth.Process() instead of just checking for not err != nil would be a viable solution?

Kind Regards

@adam-hanna
Copy link
Owner

Sorry, I'm just seeing this, now. Thanks for the report.

@areYouLazy
Copy link

The same error seems to be triggered if the X-CSRF-Token does not match the value in JWTs

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

No branches or pull requests

3 participants