Skip to content

Commit ca23a8d

Browse files
joegallojfreden
authored andcommitted
Close exchanges in HttpClientTests (elastic#115059)
1 parent 38558b3 commit ca23a8d

File tree

2 files changed

+2
-2
lines changed

2 files changed

+2
-2
lines changed

modules/ingest-geoip/src/test/java/org/elasticsearch/ingest/geoip/HttpClientTests.java

Lines changed: 2 additions & 0 deletions
Original file line numberDiff line numberDiff line change
@@ -47,6 +47,7 @@ public static void startServer() throws Throwable {
4747
server.createContext("/404/", exchange -> {
4848
try {
4949
exchange.sendResponseHeaders(404, 0);
50+
exchange.close();
5051
} catch (Exception e) {
5152
fail(e);
5253
}
@@ -102,6 +103,7 @@ public boolean checkCredentials(String username, String password) {
102103
exchange.getResponseHeaders().add("Location", "/" + destination + "/");
103104
}
104105
exchange.sendResponseHeaders(302, 0);
106+
exchange.close();
105107
} catch (Exception e) {
106108
fail(e);
107109
}

muted-tests.yml

Lines changed: 0 additions & 2 deletions
Original file line numberDiff line numberDiff line change
@@ -318,8 +318,6 @@ tests:
318318
- class: org.elasticsearch.xpack.inference.rest.ServerSentEventsRestActionListenerTests
319319
method: testNoStream
320320
issue: https://github.com/elastic/elasticsearch/issues/114788
321-
- class: org.elasticsearch.ingest.geoip.HttpClientTests
322-
issue: https://github.com/elastic/elasticsearch/issues/112618
323321
- class: org.elasticsearch.xpack.remotecluster.RemoteClusterSecurityWithApmTracingRestIT
324322
method: testTracingCrossCluster
325323
issue: https://github.com/elastic/elasticsearch/issues/112731

0 commit comments

Comments
 (0)