-
Notifications
You must be signed in to change notification settings - Fork 220
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
Handle self user registration failure when the email domain is not matched with organization email domain #813
Conversation
Better if we can have validation logic at the portal level so that self registering with incorrect email domain can be blocked and allowed to set an allowed domain. But raising the PR as the previous server error and the display msg looks not intuitive for the end user. |
...tity.recovery/src/main/java/org/wso2/carbon/identity/recovery/IdentityRecoveryConstants.java
Outdated
Show resolved
Hide resolved
PR builder started |
PR builder completed |
…tched with organization email domain
da9ce97
810797c
to
da9ce97
Compare
PR builder started |
Codecov ReportAttention: Patch coverage is
Additional details and impacted files@@ Coverage Diff @@
## master #813 +/- ##
============================================
+ Coverage 24.21% 24.72% +0.50%
- Complexity 996 1012 +16
============================================
Files 267 267
Lines 15478 15492 +14
Branches 2094 2098 +4
============================================
+ Hits 3748 3830 +82
+ Misses 11256 11168 -88
- Partials 474 494 +20 ☔ View full report in Codecov by Sentry. |
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/11143127704
14529e1
to
0b9ba6f
Compare
2fcf101
to
f299545
Compare
Proposed changes in this pull request
Set email domain organization discovery for an organization.
Try self registering with setting email as not matching email domain
Related Issues
After the fix