-
Notifications
You must be signed in to change notification settings - Fork 372
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
Integrate IdentityKeyStoreResolver #2524
Conversation
...ntity.oauth/src/main/java/org/wso2/carbon/identity/oauth2/authcontext/JWTTokenGenerator.java
Show resolved
Hide resolved
....wso2.carbon.identity.oauth/src/main/java/org/wso2/carbon/identity/oauth2/util/JWTUtils.java
Show resolved
Hide resolved
...so2.carbon.identity.oauth/src/main/java/org/wso2/carbon/identity/oauth2/util/OAuth2Util.java
Outdated
Show resolved
Hide resolved
...so2.carbon.identity.oauth/src/main/java/org/wso2/carbon/identity/oauth2/util/OAuth2Util.java
Outdated
Show resolved
Hide resolved
...so2.carbon.identity.oauth/src/main/java/org/wso2/carbon/identity/oauth2/util/OAuth2Util.java
Show resolved
Hide resolved
...so2.carbon.identity.oauth/src/main/java/org/wso2/carbon/identity/oauth2/util/OAuth2Util.java
Show resolved
Hide resolved
...so2.carbon.identity.oauth/src/main/java/org/wso2/carbon/identity/oauth2/util/OAuth2Util.java
Outdated
Show resolved
Hide resolved
How we plan to handle token-exchange/Org switch when we have different keystores, we have to update those source code also right? |
c5c0c5f
to
da5f028
Compare
da5f028
to
ffd7c83
Compare
Codecov ReportAttention: Patch coverage is
Additional details and impacted files@@ Coverage Diff @@
## master #2524 +/- ##
============================================
- Coverage 56.12% 55.72% -0.41%
- Complexity 8092 8227 +135
============================================
Files 632 632
Lines 46016 47000 +984
Branches 7520 7767 +247
============================================
+ Hits 25827 26191 +364
- Misses 16472 17000 +528
- Partials 3717 3809 +92
Flags with carried forward coverage won't be shown. Click here to find out more. ☔ View full report in Codecov by Sentry. |
Please link the doc PR which includes the supported flows in oAuth. |
Created the doc PR |
Purpose
IdentityKeyStoreResolver
class into the OAuth flow.Important