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

Screen value (mobile number) is not visible in the UI for Federated SMS OTP connector #20432

Closed
JayaShakthi97 opened this issue May 21, 2024 · 2 comments
Assignees
Labels
Affected/7.0.0 frontend Resolution/Fixed Type/Bug ui Issues reported in the user interface of the embedded portals (Ex: Myaccount, Console)
Milestone

Comments

@JayaShakthi97
Copy link
Contributor

Describe the issue:

When trying SMS OTP flow with Federated SMS OTP connector, the screen value is not visible in the UI even though the configurations are added.

How to reproduce:

  1. Setup federated SMS OTP through the management console. (You have to enable the identity providers menu)
  2. Add it to an application's login flow as the second factor.
  3. Add the following config to the deployment.toml
[authentication.authenticator.sms_otp.parameters]
screenUserAttribute = "http://wso2.org/claims/mobile"
noOfDigits = 4
  1. Restart the server and try login flow with the application.
  2. Observe the OTP verification page.

Expected behavior:

Masked mobile number should be visible in the OTP verification page.

Environment information (Please complete the following information; remove any unnecessary fields) :

  • Product Version: IS 7.0 and latest
  • OS: Mac
  • Database: H2
@JayaShakthi97 JayaShakthi97 added this to the 7.1.0-m1 milestone May 21, 2024
@JayaShakthi97 JayaShakthi97 added frontend ui Issues reported in the user interface of the embedded portals (Ex: Myaccount, Console) labels May 21, 2024
@JayaShakthi97
Copy link
Contributor Author

Analysis

  • In smsOtp.jsp, it is looking for the query param named screenValue[1]. This is expected for local SMS OTP authenticator.
  • But for federated SMS OTP connector, the query param should be screenvalue[2].
  • SMS OTP page should be updated to support both.

[1] https://github.com/wso2/identity-apps/blob/f4da291e5282fed1ffa15419f6f91d411e70e7ef/identity-apps-core/apps/authentication-portal/src/main/webapp/smsOtp.jsp#L161
[2] https://github.com/wso2-extensions/identity-outbound-auth-sms-otp/blob/6ba45753b2e02080391fb55f6ea5e534b9bd86ca/component/authenticator/src/main/java/org/wso2/carbon/identity/authenticator/smsotp/SMSOTPConstants.java#L137

@JayaShakthi97
Copy link
Contributor Author

Support for both authenticators will be given with;

Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
Affected/7.0.0 frontend Resolution/Fixed Type/Bug ui Issues reported in the user interface of the embedded portals (Ex: Myaccount, Console)
Projects
None yet
Development

No branches or pull requests

1 participant