Resolve PR comments

wso2 · Dec 20, 2024 · 8304a92 · 8304a92
1 parent ef22366
commit 8304a92
Show file tree

Hide file tree

Showing 2 changed files with 2 additions and 2 deletions.
diff --git a/...ator/consent/extensions/ciba/authenticator/weblink/CIBAWebLinkAuthenticatorConstants.java b/...ator/consent/extensions/ciba/authenticator/weblink/CIBAWebLinkAuthenticatorConstants.java
@@ -33,5 +33,6 @@ public class CIBAWebLinkAuthenticatorConstants {
     public static final String USER_INFO = "userinfo";
     public static final String CLAIMS = "claims";
     public static final String AUTHORIZE_URL_PATH = "/oauth2/authorize?";
+    public static final String LOGIN_HINT = "login_hint";
 
 }
diff --git a/.../consent/extensions/ciba/authenticator/weblink/CIBAWebLinkAuthenticatorExtensionImpl.java b/.../consent/extensions/ciba/authenticator/weblink/CIBAWebLinkAuthenticatorExtensionImpl.java
@@ -39,7 +39,6 @@ public class CIBAWebLinkAuthenticatorExtensionImpl implements CIBAWebLinkAuthent
     private static final Log log = LogFactory.getLog(CIBAWebLinkAuthenticatorExtensionImpl.class);
     private static final ConsentCoreService consentCoreService =
             ConsentExtensionsDataHolder.getInstance().getConsentCoreService();
-    public static final String LOGIN_HINT = "login_hint";
 
     /**
      * Method to identify the user/users involved in the authentication.
@@ -50,7 +49,7 @@ public class CIBAWebLinkAuthenticatorExtensionImpl implements CIBAWebLinkAuthent
     @Override
     public List<AuthenticatedUser> getAuthenticatedUsers(HttpServletRequest request) {
 
-        return Arrays.stream(request.getParameter(LOGIN_HINT)
+        return Arrays.stream(request.getParameter(CIBAWebLinkAuthenticatorConstants.LOGIN_HINT)
                         .split(","))
                 .map(String::trim)
                 .map(AuthenticatedUser::createLocalAuthenticatedUserFromSubjectIdentifier)