diff options
author | Harald Musum <musum@verizonmedia.com> | 2022-08-17 10:38:48 +0200 |
---|---|---|
committer | GitHub <noreply@github.com> | 2022-08-17 10:38:48 +0200 |
commit | 5e86d125fe6a6a4247c01912caae4b2ca2acd629 (patch) | |
tree | 6b9a05bd3522212b6f799f4df66b4920a409f0c6 /configserver | |
parent | 5f57dc00592552a29cdab9c535ff05b03db08a5b (diff) |
Revert "Add session id to response for prepare and activate requests [run-systemtest]"
Diffstat (limited to 'configserver')
5 files changed, 16 insertions, 15 deletions
diff --git a/configserver/src/main/java/com/yahoo/vespa/config/server/http/v2/response/SessionActiveResponse.java b/configserver/src/main/java/com/yahoo/vespa/config/server/http/v2/response/SessionActiveResponse.java index 0c8c8946cbc..55e640d4ec7 100644 --- a/configserver/src/main/java/com/yahoo/vespa/config/server/http/v2/response/SessionActiveResponse.java +++ b/configserver/src/main/java/com/yahoo/vespa/config/server/http/v2/response/SessionActiveResponse.java @@ -18,7 +18,6 @@ public class SessionActiveResponse extends SlimeJsonResponse { Cursor root = metaData.get(); root.setString("tenant", tenantName.value()); - root.setString("session-id", Long.toString(sessionId)); root.setString("message", message); root.setString("url", "http://" + request.getHost() + ":" + request.getPort() + "/application/v2/tenant/" + tenantName + diff --git a/configserver/src/main/java/com/yahoo/vespa/config/server/http/v2/response/SessionPrepareAndActivateResponse.java b/configserver/src/main/java/com/yahoo/vespa/config/server/http/v2/response/SessionPrepareAndActivateResponse.java index 617df3868ab..e436675fb59 100644 --- a/configserver/src/main/java/com/yahoo/vespa/config/server/http/v2/response/SessionPrepareAndActivateResponse.java +++ b/configserver/src/main/java/com/yahoo/vespa/config/server/http/v2/response/SessionPrepareAndActivateResponse.java @@ -25,7 +25,6 @@ public class SessionPrepareAndActivateResponse extends SlimeJsonResponse { Cursor root = slime.get(); root.setString("tenant", tenantName.value()); - root.setString("session-id", Long.toString(result.sessionId())); root.setString("url", "http://" + request.getHost() + ":" + request.getPort() + "/application/v2/tenant/" + tenantName + "/application/" + applicationId.application().value() + diff --git a/configserver/src/main/java/com/yahoo/vespa/config/server/http/v2/response/SessionPrepareResponse.java b/configserver/src/main/java/com/yahoo/vespa/config/server/http/v2/response/SessionPrepareResponse.java index ac745f833f9..563f50d0012 100644 --- a/configserver/src/main/java/com/yahoo/vespa/config/server/http/v2/response/SessionPrepareResponse.java +++ b/configserver/src/main/java/com/yahoo/vespa/config/server/http/v2/response/SessionPrepareResponse.java @@ -31,7 +31,6 @@ public class SessionPrepareResponse extends SlimeJsonResponse { Cursor root = deployLog.get().type() != Type.NIX ? deployLog.get() : deployLog.setObject(); root.setString("tenant", tenantName.value()); - root.setString("session-id", Long.toString(sessionId)); root.setString("activate", "http://" + request.getHost() + ":" + request.getPort() + "/application/v2/tenant/" + tenantName.value() + "/session/" + sessionId + "/active"); root.setString("message", "Session " + sessionId + " for tenant '" + tenantName.value() + "' prepared."); diff --git a/configserver/src/test/java/com/yahoo/vespa/config/server/http/v2/SessionActiveHandlerTest.java b/configserver/src/test/java/com/yahoo/vespa/config/server/http/v2/SessionActiveHandlerTest.java index 1c71ef0b7fb..8638a29cf75 100644 --- a/configserver/src/test/java/com/yahoo/vespa/config/server/http/v2/SessionActiveHandlerTest.java +++ b/configserver/src/test/java/com/yahoo/vespa/config/server/http/v2/SessionActiveHandlerTest.java @@ -155,10 +155,7 @@ public class SessionActiveHandlerTest { private void assertActivationMessageOK(ActivateRequest activateRequest, String message) throws IOException { ByteArrayOutputStream byteArrayOutputStream = new ByteArrayOutputStream(); new JsonFormat(true).encode(byteArrayOutputStream, activateRequest.getMetaData().getSlime()); - long sessionId = activateRequest.getSessionId(); - assertTrue(message.contains("\"tenant\":\"" + tenantName)); - assertTrue(message.contains("\"session-id\":\"" + sessionId)); - assertTrue(message.contains("\"message\":\"Session " + sessionId + activatedMessage)); + assertTrue(message.contains("\"tenant\":\"" + tenantName + "\",\"message\":\"Session " + activateRequest.getSessionId() + activatedMessage)); assertTrue(message.contains("/application/v2/tenant/" + tenantName + "/application/" + appName + "/environment/" + "prod" + diff --git a/configserver/src/test/java/com/yahoo/vespa/config/server/http/v2/SessionPrepareHandlerTest.java b/configserver/src/test/java/com/yahoo/vespa/config/server/http/v2/SessionPrepareHandlerTest.java index 2b07cffffce..66da009946e 100644 --- a/configserver/src/test/java/com/yahoo/vespa/config/server/http/v2/SessionPrepareHandlerTest.java +++ b/configserver/src/test/java/com/yahoo/vespa/config/server/http/v2/SessionPrepareHandlerTest.java @@ -61,6 +61,7 @@ public class SessionPrepareHandlerTest extends SessionHandlerTest { private ConfigserverConfig configserverConfig; private String preparedMessage = " prepared.\"}"; + private String tenantMessage = ""; private TenantRepository tenantRepository; @Rule @@ -89,6 +90,7 @@ public class SessionPrepareHandlerTest extends SessionHandlerTest { .build(); pathPrefix = "/application/v2/tenant/" + tenant + "/session/"; preparedMessage = " for tenant '" + tenant + "' prepared.\""; + tenantMessage = ",\"tenant\":\"" + tenant + "\""; } @Test @@ -121,16 +123,13 @@ public class SessionPrepareHandlerTest extends SessionHandlerTest { } @Test - public void require_that_response_has_all_fields() throws Exception { + public void require_that_activate_url_is_returned_on_success() throws Exception { long sessionId = createSession(applicationId()); HttpResponse response = request(HttpRequest.Method.PUT, sessionId); assertNotNull(response); assertEquals(OK, response.getStatus()); - assertResponseContains(response, "\"activate\":\"http://foo:1337" + pathPrefix + sessionId + "/active\""); - assertResponseContains(response, "\"message\":\"Session " + sessionId + preparedMessage); - assertResponseContains(response, "\"tenant\":\"" + tenant + "\""); - assertResponseContains(response, "\"session-id\":\"" + sessionId + "\""); - assertResponseContains(response, "\"log\":[]"); + assertResponseContains(response, "\"activate\":\"http://foo:1337" + pathPrefix + sessionId + + "/active\",\"message\":\"Session " + sessionId + preparedMessage); } @Test @@ -187,6 +186,15 @@ public class SessionPrepareHandlerTest extends SessionHandlerTest { } @Test + public void require_that_tenant_is_in_response() throws Exception { + long sessionId = createSession(applicationId()); + HttpResponse response = request(HttpRequest.Method.PUT, sessionId); + assertNotNull(response); + assertEquals(OK, response.getStatus()); + assertResponseContains(response, tenantMessage); + } + + @Test public void require_that_preparing_with_multiple_tenants_work() throws Exception { SessionHandler handler = createHandler(); @@ -299,8 +307,7 @@ public class SessionPrepareHandlerTest extends SessionHandlerTest { } private static void assertResponseContains(HttpResponse response, String string) throws IOException { - String s = SessionHandlerTest.getRenderedString(response); - assertTrue(s, s.contains(string)); + assertTrue(SessionHandlerTest.getRenderedString(response).contains(string)); } private static void assertResponseNotContains(HttpResponse response, String string) throws IOException { |