Close exchanges in HttpClientTests (elastic#115059)

joegallo · Oct 18, 2024 · a62d495 · a62d495
1 parent 9e7d3a7
commit a62d495
Show file tree

Hide file tree

Showing 2 changed files with 2 additions and 2 deletions.
diff --git a/modules/ingest-geoip/src/test/java/org/elasticsearch/ingest/geoip/HttpClientTests.java b/modules/ingest-geoip/src/test/java/org/elasticsearch/ingest/geoip/HttpClientTests.java
@@ -47,6 +47,7 @@ public static void startServer() throws Throwable {
         server.createContext("/404/", exchange -> {
             try {
                 exchange.sendResponseHeaders(404, 0);
+                exchange.close();
             } catch (Exception e) {
                 fail(e);
             }
@@ -102,6 +103,7 @@ public boolean checkCredentials(String username, String password) {
                     exchange.getResponseHeaders().add("Location", "/" + destination + "/");
                 }
                 exchange.sendResponseHeaders(302, 0);
+                exchange.close();
             } catch (Exception e) {
                 fail(e);
             }

diff --git a/muted-tests.yml b/muted-tests.yml
@@ -393,8 +393,6 @@ tests:
 - class: org.elasticsearch.smoketest.DocsClientYamlTestSuiteIT
   method: test {yaml=reference/esql/esql-across-clusters/line_197}
   issue: https://github.com/elastic/elasticsearch/issues/114947
-- class: org.elasticsearch.ingest.geoip.HttpClientTests
-  issue: https://github.com/elastic/elasticsearch/issues/112618
 - class: org.elasticsearch.xpack.enrich.EnrichRestIT
   method: test {p0=enrich/50_data_stream/enrich documents over _bulk via a data stream}
   issue: https://github.com/elastic/elasticsearch/issues/114769