Sanitize cql in Apache Camel instrumentation #3717

anosek-an · 2021-07-29T12:23:31Z

cql is sanitized using SqlStatementSanitizer
cassandra db statement is no longer retrieved from URI - we didn't retrieve it in case of sql component anyway, and the retrieved cql statement was hardly readable
resolves Mask DB statement in Apache Camel instrumentation #1499

...roovy/io/opentelemetry/javaagent/instrumentation/apachecamel/decorators/CassandraTest.groovy

sanitize cassandra

1854460

anosek-an requested review from anuraaga, iNikem, jkwatson, laurit, mateuszrzeszutek, pavolloffay, trask and tylerbenson as code owners July 29, 2021 12:23

mateuszrzeszutek reviewed Jul 29, 2021

View reviewed changes

...roovy/io/opentelemetry/javaagent/instrumentation/apachecamel/decorators/CassandraTest.groovy Outdated Show resolved Hide resolved

use SemanticAttributes

a5efd1e

mateuszrzeszutek approved these changes Jul 29, 2021

View reviewed changes

trask approved these changes Jul 29, 2021

View reviewed changes

...roovy/io/opentelemetry/javaagent/instrumentation/apachecamel/decorators/CassandraTest.groovy Show resolved Hide resolved

trask merged commit 117891a into open-telemetry:main Jul 29, 2021

anosek-an mentioned this pull request Aug 25, 2021

REQUEST: New membership for Enkelian open-telemetry/community#817

Closed

6 tasks

anosek-an deleted the camel-sanitize-cassandra branch September 14, 2021 07:34

Provide feedback

Saved searches

Use saved searches to filter your results more quickly

Sanitize cql in Apache Camel instrumentation #3717

Sanitize cql in Apache Camel instrumentation #3717

anosek-an commented Jul 29, 2021 •

edited

Loading

Sanitize cql in Apache Camel instrumentation #3717

Sanitize cql in Apache Camel instrumentation #3717

Conversation

anosek-an commented Jul 29, 2021 • edited Loading

anosek-an commented Jul 29, 2021 •

edited

Loading