-
Notifications
You must be signed in to change notification settings - Fork 1.6k
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
Authentication docs could do with a note around .NET 8 changes #1876
Comments
Hi Bryan! Thanks for reporting this issue and letting us know about these changes in claims mapping! Ideally, this issue ticket can be closed by a PR with acceptance tests or develop a sample to merge. Do you have an intention to work on this issue being created by you? |
@devsbryan Could you help us to upgrade docs plz? |
@raman-m Can I do it? I would first update the docs, then develop a sample as you suggested earlier |
@jvanderlei Welcome! You are assigned! To start development please fork the repository to your account! Currently I don't see Ocelot repo in your account repos. |
Please be aware that we have a strict deadline to complete the .NET 9 milestone tasks. How soon can you open a pull request? |
I'll send the PR for the documentation today, I'm now just reading the details on the breaking changes. |
@devsbryan Hello, Simon! Are you online? |
@jvanderlei Please note that the issue has been marked with a label. Therefore it would be advisable to write actual unit & acceptance tests to demonstrate the issue. Please include them in the current open PR. |
@raman-m All right. I'm not the most versed dev on acceptance testing, but I'll try my best to deliver it until tomorrow. |
Please don't rush! Please relax this weekend. You may return to writing tests on Monday.
Very well. Could you help us register an Ocelot native account for testing from both local and CircleCI environments? We need your instructions on that... |
Perfect. And yes, I can. I'll look some more about the registering for CI pipeline, and return to you about this process. |
Docs Update Suggestion
The section in the authentication docs that deals with overriding the mapping of the
scp
claim no longer applies in .net 8.The
JwtSecurityTokenHandler
is no longer the default, and has been replaced byJsonWebTokenHandler
The example in the docs will work in .net 8 by simply replacing the reference to the
JwtSecurityTokenHandler
.i.e.
It is perhaps also worth mentioning in the docs that this does not solely apply to using Okta (in my case it also applies to Azure AD)
References:
Motivation for New Feature
Steps to Reproduce the Problem
n/a
Specifications
n/a
The text was updated successfully, but these errors were encountered: