Do not set content length to zero

Fixes: SeleniumHQ#11342
krmahadevan · Dec 13, 2022 · dcabb08 · dcabb08
1 parent 49aff86
commit dcabb08
Show file tree

Hide file tree

Showing 6 changed files with 0 additions and 6 deletions.
diff --git a/java/src/org/openqa/selenium/docker/v1_41/GetContainerLogs.java b/java/src/org/openqa/selenium/docker/v1_41/GetContainerLogs.java
@@ -50,7 +50,6 @@ public ContainerLogs apply(ContainerId id) {
 
     HttpResponse res = client.execute(
       new HttpRequest(GET, requestUrl)
-        .addHeader("Content-Length", "0")
         .addHeader("Content-Type", "text/plain"));
     if (res.getStatus() != HTTP_OK) {
       LOG.warning("Unable to inspect container " + id);

diff --git a/java/src/org/openqa/selenium/docker/v1_41/InspectContainer.java b/java/src/org/openqa/selenium/docker/v1_41/InspectContainer.java
@@ -52,7 +52,6 @@ public ContainerInfo apply(ContainerId id) {
 
     HttpResponse res = client.execute(
       new HttpRequest(GET, String.format("/v%s/containers/%s/json", DOCKER_API_VERSION, id))
-        .addHeader("Content-Length", "0")
         .addHeader("Content-Type", "text/plain"));
     if (res.getStatus() != HTTP_OK) {
       LOG.warning("Unable to inspect container " + id);

diff --git a/java/src/org/openqa/selenium/docker/v1_41/IsContainerPresent.java b/java/src/org/openqa/selenium/docker/v1_41/IsContainerPresent.java
@@ -38,7 +38,6 @@ public boolean apply(ContainerId id) {
 
     HttpResponse res = client.execute(
       new HttpRequest(GET, String.format("/v%s/containers/%s/json", DOCKER_API_VERSION, id))
-        .addHeader("Content-Length", "0")
         .addHeader("Content-Type", "text/plain"));
 
     return res.isSuccessful();

diff --git a/java/src/org/openqa/selenium/docker/v1_41/ListImages.java b/java/src/org/openqa/selenium/docker/v1_41/ListImages.java
@@ -58,7 +58,6 @@ public Set<Image> apply(Reference reference) {
 
     // https://docs.docker.com/engine/api/v1.41/#operation/ImageList
     HttpRequest req = new HttpRequest(GET, String.format("/v%s/images/json", DOCKER_API_VERSION))
-      .addHeader("Content-Length", "0")
       .addHeader("Content-Type", JSON_UTF_8)
       .addQueryParameter("filters", JSON.toJson(filters));
 

diff --git a/java/src/org/openqa/selenium/docker/v1_41/StartContainer.java b/java/src/org/openqa/selenium/docker/v1_41/StartContainer.java
@@ -39,7 +39,6 @@ public void apply(ContainerId id) {
     throwIfNecessary(
       client.execute(
         new HttpRequest(POST, String.format("/v%s/containers/%s/start", DOCKER_API_VERSION, id))
-          .addHeader("Content-Length", "0")
           .addHeader("Content-Type", "text/plain")),
       "Unable to start container: %s",
       id);

diff --git a/java/src/org/openqa/selenium/docker/v1_41/StopContainer.java b/java/src/org/openqa/selenium/docker/v1_41/StopContainer.java
@@ -43,7 +43,6 @@ public void apply(ContainerId id, Duration timeout) {
 
     String requestUrl = String.format("/v%s/containers/%s/stop", DOCKER_API_VERSION, id);
     HttpRequest request = new HttpRequest(POST, requestUrl)
-      .addHeader("Content-Length", "0")
       .addHeader("Content-Type", "text/plain")
       .addQueryParameter("t", seconds);