Introduce app resident tenant domain extraction and get app resident tenant from pre process username #6586
Conversation
|
Codecov ReportAttention: Patch coverage is
❌ Your patch check has failed because the patch coverage (0.00%) is below the target coverage (80.00%). You can increase the patch coverage or adjust the target coverage. Additional details and impacted files@@ Coverage Diff @@
## master #6586 +/- ##
============================================
+ Coverage 46.81% 47.75% +0.93%
- Complexity 15707 16537 +830
============================================
Files 1800 1824 +24
Lines 109453 114103 +4650
Branches 20205 21446 +1241
============================================
+ Hits 51236 54485 +3249
- Misses 51024 52209 +1185
- Partials 7193 7409 +216
Flags with carried forward coverage won't be shown. Click here to find out more. ☔ View full report in Codecov by Sentry. 🚀 New features to boost your workflow:
|
.../java/org/wso2/carbon/identity/application/authentication/framework/util/FrameworkUtils.java
Outdated
Show resolved
Hide resolved
…tenant from pre process username
ShanChathusanda93
force-pushed
the
app-resident-pre-process-branch
branch
from
3dc9c22 to
712600d
Compare
|
| appTenantDomain = appResidentTenantDomain; | ||
| } | ||
| } catch (FrameworkException e) { | ||
| log.error("Error while getting the tenant domain of the application owner.", e); |
There was a problem hiding this comment.
What is the expected behavior when this exception is thrown
Proposed changes in this pull request