Skip to content

Commit 675afce

Browse files
authored
Rename a method to 'wrap' (#15829)
1 parent 79a251c commit 675afce

3 files changed

Lines changed: 11 additions & 16 deletions

File tree

instrumentation/java-http-client/library/README.md

Lines changed: 1 addition & 1 deletion
Original file line numberDiff line numberDiff line change
@@ -43,7 +43,7 @@ public class JavaHttpClientConfiguration {
4343

4444
//Use this HttpClient implementation for making standard http client calls.
4545
public HttpClient createTracedClient(OpenTelemetry openTelemetry) {
46-
return JavaHttpClientTelemetry.builder(openTelemetry).build().createHttpClient(createClient());
46+
return JavaHttpClientTelemetry.builder(openTelemetry).build().wrap(createClient());
4747
}
4848

4949
//your configuration of the Java HTTP Client goes here:

instrumentation/java-http-client/library/src/main/java/io/opentelemetry/instrumentation/javahttpclient/JavaHttpClientTelemetry.java

Lines changed: 9 additions & 14 deletions
Original file line numberDiff line numberDiff line change
@@ -38,22 +38,17 @@ public static JavaHttpClientTelemetryBuilder builder(OpenTelemetry openTelemetry
3838
/**
3939
* Returns an instrumented {@link HttpClient} wrapping the provided client.
4040
*
41-
* @param client the HttpClient to wrap
42-
* @return an instrumented HttpClient
43-
*/
44-
public HttpClient createHttpClient(HttpClient client) {
45-
return new OpenTelemetryHttpClient(client, instrumenter, headersSetter);
46-
}
47-
48-
/**
49-
* Returns an instrumented {@link HttpClient} wrapping the provided client.
50-
*
51-
* @param client the HttpClient to wrap
52-
* @return an instrumented HttpClient
53-
* @deprecated Use {@link #createHttpClient(HttpClient)} instead.
41+
* @param client An instance of HttpClient configured as desired.
42+
* @return a tracing-enabled {@link HttpClient}.
43+
* @deprecated Use {@link #wrap(HttpClient)} instead.
5444
*/
5545
@Deprecated
5646
public HttpClient newHttpClient(HttpClient client) {
57-
return createHttpClient(client);
47+
return wrap(client);
48+
}
49+
50+
/** Returns a new instrumented {@link HttpClient} that wraps the provided client. */
51+
public HttpClient wrap(HttpClient client) {
52+
return new OpenTelemetryHttpClient(client, instrumenter, headersSetter);
5853
}
5954
}

instrumentation/java-http-client/library/src/test/java/io/opentelemetry/instrumentation/javahttpclient/JavaHttpClientTest.java

Lines changed: 1 addition & 1 deletion
Original file line numberDiff line numberDiff line change
@@ -28,7 +28,7 @@ protected HttpClient configureHttpClient(HttpClient httpClient) {
2828
.setCapturedResponseHeaders(
2929
Collections.singletonList(AbstractHttpClientTest.TEST_RESPONSE_HEADER))
3030
.build()
31-
.createHttpClient(httpClient);
31+
.wrap(httpClient);
3232
}
3333
}
3434

0 commit comments

Comments
 (0)