Skip to content

Fix for jdbc target and duplicate headers#1647

Merged
trask merged 2 commits into
masterfrom
trask/fix-for-jdbc-and-headers
Apr 23, 2021
Merged

Fix for jdbc target and duplicate headers#1647
trask merged 2 commits into
masterfrom
trask/fix-for-jdbc-and-headers

Conversation

@trask

@trask trask commented Apr 22, 2021

Copy link
Copy Markdown
Member

Update otel fork submodule, see: #1645

Updates Jdbc smoke tests due to the new (better) target attribute.

Fixes an unrelated issue where the Jdbc smoke tests just started failing due to recent update in mysql image that requires enabling TLSv1.3 in the driver

Resolves #1640

Attaching SNAPSHOT release from this PR: applicationinsights-agent-3.0.4-BETA-SNAPSHOT.jar.zip

return 6379;
case SemanticAttributes.DbSystemValues.MARIADB:
case SemanticAttributes.DbSystemValues.MYSQL:
return 3306;

Copy link
Copy Markdown
Contributor

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

Does otel-java have constants for these ports? It will be nice if they do. Or also where is the spec for defining these port numbers?

Copy link
Copy Markdown
Member Author

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

see the comment above

@trask trask merged commit c280ec8 into master Apr 23, 2021
@trask trask deleted the trask/fix-for-jdbc-and-headers branch April 23, 2021 03:25
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Labels

None yet

Projects

None yet

Development

Successfully merging this pull request may close these issues.

Spring webclient instrumentation

2 participants