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

Token validation related security updates in previous versions #540

Closed
snekbaev opened this issue Nov 30, 2016 · 6 comments
Closed

Token validation related security updates in previous versions #540

snekbaev opened this issue Nov 30, 2016 · 6 comments

Comments

@snekbaev
Copy link

Hi,

just discovered: https://auth0.com/blog/critical-vulnerabilities-in-json-web-token-libraries and wanted to ask in which version this has been addressed, I'm using .NET 4.6.1 with System.IdentityModel.Tokens.Jwt and can't upgrade it to 5.x because is it not compatible, thus, I'm stuck with 4.0.3.308261200.

And question is: am I safe? :)

Thank you!

@brentschmaltz
Copy link
Member

@snekbaev By default we require signed tokens and do not allow 'none'.
So you are fine with 4.x or 5.x.

What issues are you having with 5.x compatibility?

@snekbaev
Copy link
Author

@brentschmaltz if I remember correctly it was
http://stackoverflow.com/questions/38119877/tokenvalidationparameters-no-longer-working-after-upgrade-to-5-0-0 and #466

Basically I'm using Katana with WebAPI 2 and apparently some crucial type was moved to a different namespace in v5 thus it doesn't work :)

@brentschmaltz
Copy link
Member

brentschmaltz commented Dec 1, 2016

@snekbaev yes, 5.x does not work with Katana. Currently we are committed to supporting 4.x and 5.x. I was just wondering if it related to Katana or some other issue.

@snekbaev
Copy link
Author

snekbaev commented Dec 1, 2016

@brentschmaltz well, wherever the issue is, one thing for sure is that it will make a lot of people happy not to have that dependency hanging in the nuget's updates with a note in a readme file saying "DO NOT UPDATE!" :)))

@brentschmaltz
Copy link
Member

@snekbaev @brockallen and others, we are aware of that this is causing headaches, we are investigating a fix. Hopefully it will show up soon.

@brentschmaltz
Copy link
Member

Issue is resolved pertaining to security risk. Back-compat is a separate issue.

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

2 participants