-
Notifications
You must be signed in to change notification settings - Fork 255
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
Security vulnerabilities #97
Comments
@sarneeh thank you for opening this issue. 👍 Security concerns applicable to JWTs are the same as for other token schemes. We should advise people using JWTs that they do not protect against OWASP attacks and are merely a convenient way of transmitting signed claims/data. 📝 |
I partially agree. You have something like a So I guess that in the end, it doesn't matter if you use I'd love to prepare some PR but I'm hard on time right now and will be free just in the next 1-2 months. If someone won't outrun me I'll prepare some PR 😄 |
As said above cookie can have httpOnly option.
|
Just a suggestion: it would be cool to add some information about security concerns in different JWT storage methods on the client-side (cookie/localStorage) - XSS, CSRF etc.
The text was updated successfully, but these errors were encountered: