edits

Author: melissaahn

common/src/main/java/com/microsoft/identity/common/internal/controllers/BrokerMsalController.java

@@ -1450,7 +1450,7 @@ public void putValueInSuccessEvent(@NonNull final ApiEndEvent event,
     /**
      * Execute web app request in broker.
      *
-     * @param request request string
+     * @param request request string.
      * @param minBrokerProtocolVersion minimum broker protocol version the caller requires.
      * @param additionalRequiredParams additional required parameters for web app request.
      * @throws BaseException
@@ -1829,7 +1829,6 @@ private BrokerInteractiveTokenCommandParameters buildInteractiveTokenParametersF
                 .authority(authority)
                 .scopes(scopeSet)
                 .loginHint(webAppsRequest.getLoginHint())
-                .claimsRequestJson(webAppsRequest.getClaims())
                 .correlationId(correlationId)
                 .requiredBrokerProtocolVersion(minBrokerProtocolVersion)
                 .prompt(prompt)

common/src/test/java/com/microsoft/identity/common/internal/controllers/BrokerMsalControllerTest.java

@@ -290,7 +290,6 @@ public void testExecuteWebAppRequest_SilentError_UiNotAllowed() throws Exception
                 "https://redirect",
                 "corr-id",
                 "login", // Setting to login
-                null,
                 false,
                 null,
                 null,
@@ -329,7 +328,6 @@ public void testExecuteWebAppRequest_SilentError_UiNotAllowed_ESTS() throws Exce
                 "https://redirect",
                 "corr-id",
                 "login", // Setting to login
-                null,
                 true,
                 null,
                 null,
@@ -420,7 +418,6 @@ private static String buildInteractiveGetTokenRequestJson(final boolean isSts) {
                 "https://demoapp.com/",
                 "corr-id",
                 "login",                               // prompt forces interactive
-                null,
                 isSts,                                  // isSecurityTokenService
                 null,
                 null,
@@ -446,7 +443,6 @@ private String buildStrictlySilentGetTokenRequestJson(final boolean isSts) throw
                 "https://demoapp.com/",  // redirectUri (required)
                 "corr-id",           // correlationId (optional)
                 "none",              // prompt ("none" for silent)
-                null,                // claims
                 isSts,               // isSecurityTokenService
                 null,                // nonce
                 null,                // state
@@ -474,7 +470,6 @@ private String buildFallbackSilentGetTokenRequestJson(final boolean isSts) throw
                 "https://demoapp.com/",  // redirectUri (required)
                 "corr-id",           // correlationId (optional)
                 "select_account",    // prompt
-                null,                // claims
                 isSts,               // isSecurityTokenService
                 null,                // nonce
                 null,                // state

common4j/src/main/com/microsoft/identity/common/java/commands/webapps/WebAppsGetTokenSubOperationRequest.kt

@@ -56,10 +56,6 @@ data class WebAppsGetTokenSubOperationRequest(
     @SerializedName(FIELD_PROMPT)
     val prompt: String? = null,
 
-    // Optional; claims request in JSON format.
-    @SerializedName(FIELD_CLAIMS)
-    val claims: String? = null,
-
     // If not provided, we assume this is false.
     @SerializedName(FIELD_IS_SECURITY_TOKEN_SERVICE)
     val isSecurityTokenService: Boolean = false,
@@ -93,7 +89,6 @@ data class WebAppsGetTokenSubOperationRequest(
         const val FIELD_REDIRECT_URI = "redirectUri"
         const val FIELD_CORRELATION_ID = "correlationId"
         const val FIELD_PROMPT = "prompt"
-        const val FIELD_CLAIMS = "claims"
         const val FIELD_IS_SECURITY_TOKEN_SERVICE = "isSts"
         const val FIELD_NONCE = "nonce"
         const val FIELD_STATE = "state"