Use application tenant domain to retrieve client app #2557
Conversation
mpmadhavig
force-pushed
the
feature-enable-disable-config-api
branch
from
d97c5b5
to
4e56039
Compare
|
PR builder started |
|
PR builder completed |
mpmadhavig
force-pushed
the
feature-enable-disable-config-api
branch
from
f51c33a
to
87dc56a
Compare
|
PR builder started |
|
PR builder completed |
mpmadhavig
force-pushed
the
feature-enable-disable-config-api
branch
from
5a92755
to
1b2aa8c
Compare
|
PR builder started |
|
PR builder completed |
There was a problem hiding this comment.
Approving the pull request based on the successful pr build https://github.com/wso2/product-is/actions/runs/10523947074
| tenantDomain); | ||
| omitUsernameInIntrospectionRespForAppTokens = oAuthApp.isOmitUsernameInIntrospectionRespForAppTokens(); | ||
|
|
||
| return Boolean.TRUE.equals(oAuthAppDO.isOmitUsernameInIntrospectionRespForAppTokens()); |
There was a problem hiding this comment.
cant we just return the oAuthAppDO.isOmitUsernameInIntrospectionRespForAppTokens()?
There was a problem hiding this comment.
We can. But to be safe used this.
There was a problem hiding this comment.
oAuthAppDO.isOmitUsernameInIntrospectionRespForAppTokens() never be null right? then its ok
Proposed changes in this pull request
When retrieving an application, use the tenant domain specified in the access token, instead of the user's tenant domain.