Skip to content
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

[Feature][Improvement] Introduce tenant-wise username input validation. #16357

Closed
Thumimku opened this issue Aug 2, 2023 · 6 comments
Closed
Assignees

Comments

@Thumimku
Copy link
Contributor

Thumimku commented Aug 2, 2023

Epic: https://github.com/wso2-enterprise/iam-engineering/issues/648
Currently IS has username validation can be set on user store level.
Email as user name can be configured also https://is.docs.wso2.com/en/latest/guides/identity-lifecycles/enable-email-as-username/

But what we need a tenant-wise username validator where the admin can enforce rule based validators such as min, max character efficiently.

@Thumimku
Copy link
Contributor Author

Postponing to alpha cause we need tenant qualified URL feature to be enabled

@Thumimku
Copy link
Contributor Author

Thumimku commented Oct 16, 2023

Integration test for REST API: #16983

@Thumimku
Copy link
Contributor Author

Thumimku commented Oct 31, 2023

We are not delievering this feature as per the mail due to a functional limitaiton,
check: [Unification] Bring new tenant level Password/username validation feature to IS

@Thumimku
Copy link
Contributor Author

@bhagyasakalanka is working on removing the feature from input validation listener.
@bhagyasakalanka can you please update the status?

@Thumimku Thumimku modified the milestones: 7.0.0-alpha, 7.0.0-alpha2 Oct 31, 2023
@bhagyasakalanka
Copy link
Contributor

Integration tests are fixed with: #17508
Username Input validation disable for IS: wso2/carbon-identity-framework#5106

@bhagyasakalanka
Copy link
Contributor

wso2/identity-apps#4442

@madurangasiriwardena madurangasiriwardena removed this from the 7.0.0-alpha2 milestone Nov 10, 2023
@Thumimku Thumimku removed their assignment Nov 13, 2023
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Projects
None yet
Development

No branches or pull requests

4 participants