search

aspnet / AspNetKatana

Microsoft's OWIN implementation, the Katana project
Apache License 2.0
963 stars 332 forks source link

Is Owin 4 compatible with Microsoft.IdentityModel 6+ #394

Open gabriellebourdages opened 3 years ago

gabriellebourdages commented 3 years ago

Hi, We're currently using Owin 4.1.0 with Microsoft.IdentityModel 5.3.0 and need to upgrade some projects to Microsoft.IdentityModel version 6.8 and I'm wondering if Owin 4.1.0 can work with Microsoft.IdentityModel 6.8.

We've updated all the packages, but we do get some exceptions like the one below when trying to configure OpenId :

Microsoft.Owin.Hosting.StartOptions 
 error: 
Method not found: 'Void Microsoft.Owin.Security.OpenIdConnect.OpenIdConnectAuthenticationNotifications.set_RedirectToIdentityProvider(System.Func`2<Microsoft.Owin.Security.Notifications.RedirectToIdentityProviderNotification`2<Microsoft.IdentityModel.Protocols.OpenIdConnect.OpenIdConnectMessage,Microsoft.Owin.Security.OpenIdConnect.OpenIdConnectAuthenticationOptions>,System.Threading.Tasks.Task>)'.

According to the release notes for Microsoft.IdentityModel, there seems to be some breaking changes, but there's no mention to the method that we see in the error message (the project builds properly too), so I'm not sure if it's due to breaking changes or configuration issues on our end.

Let me know if you need more information.

Tratcher commented 3 years ago

Someone tried this dependency update recently over at https://github.com/rzontar/pkce/tree/package-update. It seems to build and the tests pass. That should have caught any API breaks.