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

TokenValidationParameters namespace change impacts OpenIdConnectAuthenticationOptions #466

Closed
chriswill opened this issue Jul 2, 2016 · 5 comments
Labels
Customer reported Indicates issue was opened by customer Investigate We are not quite sure what the issue is. Question User has asked a question

Comments

@chriswill
Copy link

I understand that TokenValidationParameters has moved from System.IdentityModel.Tokens to Microsoft.IdentityModel.Tokens in the 5.0.0 release.

However the class Microsoft.Owin.Security.OpenIdConnect.OpenIdConnectAuthenticationOptions has a property called TokenValidationParameters which references System.IdentityModel.Tokens.TokenValidationParameters. It's current version is 3.01 and has not been updated to reflect the namespace change. The last update was Feb, 2015.

This means that 5.0.0 breaks an existing installation of Microsoft.Owin.Security.OpenIdConnect.

Are you aware of any plans to update that library? I had to roll back to get my app working again.

@tushargupta51
Copy link
Contributor

@chriswill Katana (Microsoft.Owin.Security.OpenIdConnect.OpenIdConnectAuthenticationOptions) does not support Wilson 5.x and there are no current plans to update 3.0.1 to use wilson 5.x. You need to switch to new asp.net core to get this support: https://github.com/aspnet/Security

@leastprivilege
Copy link
Contributor

That will be problematic.

I already heard reports of users where the Katana JWT bearer MW clashes with some other lib that depends on v5.

@brentschmaltz
Copy link
Member

@leastprivilege @chriswill an issue needs to be opened against Katana. @leastprivilege can you post here a link to the issue folks are seeing. I recently moved a project from Wilson4.x to Wilson5.x the changes were small.

@brentschmaltz brentschmaltz added Customer reported Indicates issue was opened by customer Investigate We are not quite sure what the issue is. Question User has asked a question labels Jul 5, 2016
@leastprivilege
Copy link
Contributor

@Tratcher
Copy link
Contributor

Tratcher commented Jul 6, 2016

Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
Customer reported Indicates issue was opened by customer Investigate We are not quite sure what the issue is. Question User has asked a question
Projects
None yet
Development

No branches or pull requests

6 participants