compile fixes

Signed-off-by: Andrew Azores <[email protected]>
cryostatio · Apr 22, 2023 · 3942f02 · 3942f02
1 parent f6098f4
commit 3942f02
Show file tree

Hide file tree

Showing 2 changed files with 17 additions and 24 deletions.
diff --git a/src/main/java/io/cryostat/net/web/http/api/v2/TargetsPostHandler.java b/src/main/java/io/cryostat/net/web/http/api/v2/TargetsPostHandler.java
@@ -180,28 +180,23 @@ public IntermediateResponse<ServiceRef> handle(RequestParameters params) throws
                             ? Optional.empty()
                             : Optional.of(new Credentials(username, password));
 
-            if (credentials.isPresent()) {
-                if (storeCredentials) {
-                    String matchExpression =
-                            CredentialsManager.targetIdToMatchExpression(connectUrl);
-                    int id = credentialsManager.addCredentials(matchExpression, credentials.get());
-                    notificationFactory
-                            .createBuilder()
-                            .metaCategory("CredentialsStored")
-                            .metaType(HttpMimeType.JSON)
-                            .message(
-                                    Map.of(
-                                            "id",
-                                            id,
-                                            "matchExpression",
-                                            matchExpression,
-                                            "numMatchingTargets",
-                                            1))
-                            .build()
-                            .send();
-                } else {
-                    credentialsManager.setSessionCredentials(uri.toString(), credentials.get());
-                }
+            if (storeCredentials && credentials.isPresent()) {
+                String matchExpression = CredentialsManager.targetIdToMatchExpression(connectUrl);
+                int id = credentialsManager.addCredentials(matchExpression, credentials.get());
+                notificationFactory
+                        .createBuilder()
+                        .metaCategory("CredentialsStored")
+                        .metaType(HttpMimeType.JSON)
+                        .message(
+                                Map.of(
+                                        "id",
+                                        id,
+                                        "matchExpression",
+                                        matchExpression,
+                                        "numMatchingTargets",
+                                        1))
+                        .build()
+                        .send();
             }
 
             String jvmId = jvmIdHelper.getJvmId(uri.toString(), false, credentials);

diff --git a/src/test/java/io/cryostat/net/web/http/api/v2/TargetsPostHandlerTest.java b/src/test/java/io/cryostat/net/web/http/api/v2/TargetsPostHandlerTest.java
@@ -478,8 +478,6 @@ void testRequestWithDryRunQueryIncludingCredentials() throws Exception {
         IntermediateResponse<ServiceRef> response = handler.handle(params);
         MatcherAssert.assertThat(response.getStatusCode(), Matchers.equalTo(202));
 
-        Mockito.verify(credentialsManager).setSessionCredentials(Mockito.any(), Mockito.any());
-
         ServiceRef respRef = response.getBody();
         MatcherAssert.assertThat(respRef.getServiceUri(), Matchers.equalTo(new URI(connectUrl)));
         MatcherAssert.assertThat(respRef.getAlias(), Matchers.equalTo(Optional.of(alias)));