-
-
Notifications
You must be signed in to change notification settings - Fork 768
Description
Is there an existing issue for this?
- I have searched the existing issues
What happened?
Reported on Discord by @sboshuis:
Hi all, we are testing some upgrades of larger applications to 10.1 . First of all great work to all involved. The multilanguage problems seem to have been solved!
We do have other issues. We use the great azure adb2c module from David (Intelequia) and that uses for example System.IdentityModel.Tokens.Jwt 6.12.2
In the DNN release I see 2 different things related to this:
Replaced legacy System.IdentityModel.Tokens.Jwt #6356 Thanks @bdukes
Bumped System.IdentityModel.Tokens.Jwt from 4.0.2.206221351 to 6.31.0 #5694 Thanks @bdukesI think the replaced one won from the bumbed one 🙂 because after the upgrade to 10.1 the DLL is gone from bin folder. In the web.config there is still a bindingredirect and thus it errors.
for now placing back the dll solved it but looking at Bumped System.IdentityModel.Tokens.Jwt from 4.0.2.206221351 to 6.31.0 #5694 Thanks @bdukes
maybe we should upgrade it anyways ourself manually? Or contact David and see if they can also switch to the Microsoft one as the System. is now marked as legacy
But for anyone using this module: after upgrade copy back the System.IdentityModel.Tokens.Jwt.dll to your bin folder and your good 🙂
Steps to reproduce?
- Install Azure AD B2C module
- Install JWT auth provider
- Upgrade to DNN 10
Current Behavior
Required DLL is removed
Expected Behavior
DLL remains
Affected Versions
10.0.1 (latest v10 release)
Code of Conduct
- I agree to follow this project's Code of Conduct