From edee76680ac9663b65ebd50370f07de18395cc15 Mon Sep 17 00:00:00 2001 From: Andriy Redko Date: Fri, 17 May 2024 15:36:16 -0400 Subject: [PATCH] Address code review comments Signed-off-by: Andriy Redko --- .../zipkin2/storage/elasticsearch/OpensearchContainer.java | 6 +----- 1 file changed, 1 insertion(+), 5 deletions(-) diff --git a/opensearch/src/test/java/zipkin2/storage/elasticsearch/OpensearchContainer.java b/opensearch/src/test/java/zipkin2/storage/elasticsearch/OpensearchContainer.java index 569ac64..6060176 100644 --- a/opensearch/src/test/java/zipkin2/storage/elasticsearch/OpensearchContainer.java +++ b/opensearch/src/test/java/zipkin2/storage/elasticsearch/OpensearchContainer.java @@ -47,11 +47,7 @@ class OpensearchContainer extends GenericContainer { ElasticsearchStorage.Builder newStorageBuilder() { WebClientBuilder builder = WebClient.builder("http://" + hostPort()) - // Elasticsearch 7 never returns a response when receiving an HTTP/2 preface instead of the - // more valid behavior of returning a bad request response, so we can't use the preface. - // - // TODO: find or raise a bug with Elastic - .factory(ClientFactory.builder().useHttp2Preface(false).build()); + .factory(ClientFactory.builder().build()); builder.decorator((delegate, ctx, req) -> { final HttpResponse response = delegate.execute(ctx, req); return HttpResponse.of(response.aggregate().thenApply(r -> {