wso2 / product-is

Welcome to the WSO2 Identity Server source code! For info on working with the WSO2 Identity Server repository and contributing code, click the link below.
http://wso2.github.io/
Apache License 2.0
727 stars 712 forks source link

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

Open JayaShakthi97 opened 1 month ago

JayaShakthi97 commented 1 month ago

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
  4. Restart the server and try login flow with the application.
  5. 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) :

JayaShakthi97 commented 1 month ago

Analysis

[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 commented 1 month ago

Support for both authenticators will be given with;