Skip to content

Commit 1afddbc

Browse files
committed
Add getServerAddress and getServerPort implementations to VertxSqlClientAttributesGetter
SqlClientAttributesGetter extends ServerAttributesGetter which requires getServerAddress and getServerPort implementations for capturing server attributes at request time. VertxSqlClientRequest already has getHost() and getPort() methods that extract values from SqlConnectOptions, so we can implement these methods to properly capture server.address and server.port attributes.
1 parent 4b6f289 commit 1afddbc

File tree

1 file changed

+12
-0
lines changed

1 file changed

+12
-0
lines changed

instrumentation/vertx/vertx-sql-client/vertx-sql-client-common/javaagent/src/main/java/io/opentelemetry/javaagent/instrumentation/vertx/sql/VertxSqlClientAttributesGetter.java

Lines changed: 12 additions & 0 deletions
Original file line numberDiff line numberDiff line change
@@ -41,6 +41,18 @@ public String getDbNamespace(VertxSqlClientRequest request) {
4141
return request.getDatabase();
4242
}
4343

44+
@Nullable
45+
@Override
46+
public String getServerAddress(VertxSqlClientRequest request) {
47+
return request.getHost();
48+
}
49+
50+
@Nullable
51+
@Override
52+
public Integer getServerPort(VertxSqlClientRequest request) {
53+
return request.getPort();
54+
}
55+
4456
@Override
4557
public Collection<String> getRawQueryTexts(VertxSqlClientRequest request) {
4658
return singleton(request.getQueryText());

0 commit comments

Comments
 (0)