feat: update apache-httpclient to not use HttpEntity getContent (#410)

* update apache-httpclient to not use getContent * spotless apply
hypertrace · Dec 31, 2024 · 40c54bf · 40c54bf
1 parent aca5f12
commit 40c54bf
Showing 1 changed file with 5 additions and 1 deletion.
diff --git a/...try/javaagent/instrumentation/hypertrace/apachehttpclient/v4_0/ApacheHttpClientUtils.java b/...try/javaagent/instrumentation/hypertrace/apachehttpclient/v4_0/ApacheHttpClientUtils.java
@@ -19,6 +19,7 @@
 import io.opentelemetry.api.common.AttributeKey;
 import io.opentelemetry.api.trace.Span;
 import io.opentelemetry.javaagent.instrumentation.hypertrace.apachehttpclient.v4_0.ApacheHttpClientObjectRegistry.SpanAndAttributeKey;
+import java.io.ByteArrayInputStream;
 import java.io.IOException;
 import java.io.InputStream;
 import java.io.InputStreamReader;
@@ -113,7 +114,10 @@ public static void traceEntity(
             && contentEncoding.getValue().toLowerCase().contains("gzip");
     if (entity.isRepeatable()) {
       try {
-        InputStream contentStream = entity.getContent();
+        BoundedByteArrayOutputStream byteArrayOutputStream =
+            BoundedBuffersFactory.createStream(charset);
+        entity.writeTo(byteArrayOutputStream);
+        InputStream contentStream = new ByteArrayInputStream(byteArrayOutputStream.toByteArray());
         if (isGzipEncoded) {
           try {
             contentStream = new GZIPInputStream(contentStream);