-
Notifications
You must be signed in to change notification settings - Fork 378
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
Allow sub organization applications to issue tokens to access the resources in sub organizations #2616
Allow sub organization applications to issue tokens to access the resources in sub organizations #2616
Conversation
98ad3c9
to
ecb8ca8
Compare
Please take one of the following actions:
Your prompt attention to this matter is greatly appreciated. Thank you for your understanding and collaboration! 🙏 |
This PR is WIP and will be completed withing next couple of days. |
08e2154
to
5b0bb86
Compare
...on.identity.oauth/src/main/java/org/wso2/carbon/identity/oauth2/token/AccessTokenIssuer.java
Outdated
Show resolved
Hide resolved
...java/org/wso2/carbon/identity/oauth2/client/authentication/BasicAuthClientAuthenticator.java
Show resolved
Hide resolved
....wso2.carbon.identity.oauth/src/main/java/org/wso2/carbon/identity/oauth2/OAuth2Service.java
Show resolved
Hide resolved
...y.oauth/src/main/java/org/wso2/carbon/identity/oauth2/validators/TokenValidationHandler.java
Show resolved
Hide resolved
5b0bb86
to
388a274
Compare
ff9bac9
to
1148fcb
Compare
....wso2.carbon.identity.oauth/src/main/java/org/wso2/carbon/identity/oauth2/OAuth2Service.java
Show resolved
Hide resolved
21693aa
to
cd494b5
Compare
PR builder started |
PR builder completed |
PR builder started |
PR builder completed |
PR builder started |
.../wso2/carbon/identity/oauth2/token/handlers/grant/AbstractAuthorizationGrantHandlerTest.java
Show resolved
Hide resolved
...test/java/org/wso2/carbon/identity/oauth2/token/handlers/grant/PasswordGrantHandlerTest.java
Show resolved
Hide resolved
...n.identity.oauth/src/test/java/org/wso2/carbon/identity/oauth2/token/JWTTokenIssuerTest.java
Outdated
Show resolved
Hide resolved
...2.carbon.identity.oauth/src/test/java/org/wso2/carbon/identity/oauth2/OAuth2ServiceTest.java
Outdated
Show resolved
Hide resolved
...n.identity.oauth/src/test/java/org/wso2/carbon/identity/oauth2/token/JWTTokenIssuerTest.java
Outdated
Show resolved
Hide resolved
...n.identity.oauth/src/test/java/org/wso2/carbon/identity/oauth2/token/JWTTokenIssuerTest.java
Outdated
Show resolved
Hide resolved
PR builder completed |
cd494b5
to
06b64b5
Compare
…ources in sub organizations
06b64b5
to
d70822e
Compare
PR builder started |
PR builder completed |
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
Approving the pull request based on the successful pr build https://github.com/wso2/product-is/actions/runs/12974134655
Create a separate issue for unit tests. Based on integration test pass status and manual testing, approving the PR |
Proposed changes in this pull request
/t/{tenant-domain}/o/{org-id}/oauth2/token?scope=openid...
When should this PR be merged
[1] #2578