* http.url must not contain credentials
* remove user info from url
* Update HttpClientTracerTest.groovy
fix inconsistency in protocol
* fix httpClient and httpServer
* Update HttpServerTracer.java
* Only scrub userinfo if present
Co-authored-by: Anuraag Agrawal <aanuraag@amazon.co.jp>
* Instrumenter API improvements
* Move HTTP & net classes to separate packages
* Remove `db` prefix from method names in `DbAttributesExtractor`
* Add request-only net attributes extractor (it'll be needed in spring-sleuth-otel once we decide to use Instrumenters there)
* One NetAttributesExtractor class, javadocs
* add missing @Nullable
* Apply suggestions from code review
Co-authored-by: Trask Stalnaker <trask.stalnaker@gmail.com>
* spotless
Co-authored-by: Trask Stalnaker <trask.stalnaker@gmail.com>
* Use Instrumenter in JMS instrumentation
And introduce messaging semantic conventions
* Code review comments
* Use Instrumenter in JMS instrumentation - code review comments
* remove useless test
* fix missing instrumentation version
* Code review comments
* Do not allow conditional delegation to SDK
* Use Instrumenter in JDBC instrumentation
Except jdbc-datasource, that one is going to be in a separate PR
* Use Instrumenter in JDBC instrumentation - code review comments
* Record internal metric for SQL cache misses
And use `SupportabilityMetrics` in `Instrumenter`
* Fix broken shouldStart() logic
* Code review comments
* Instrumenter
* Spotless
* More instrumenter
* More
* Instrumenter API
* New Instrumenter API - InstrumenterBuilder (#20)
* New Instrumenter API - InstrumenterBuilder
* New Instrumenter API - InstrumenterBuilder - code review comments
* Start doccing
* Docs
* Most tests
* Last test
* Stick with current hostName lookup for now
* Update testing-common/src/main/groovy/io/opentelemetry/instrumentation/test/base/HttpClientTest.groovy
Co-authored-by: Mateusz Rzeszutek <mrzeszutek@splunk.com>
* New Instrumenter API - code review comments (#21)
* Doc
* Checkstyle
* Cleanups
Co-authored-by: Mateusz Rzeszutek <mrzeszutek@splunk.com>
* Move and rename NetPeerUtils
* Rename NetPeerUtils to NetPeerAttributes; inject instance instead of using a global var
* Rename SpanAttributeSetter to AttributeSetter (might be used to set on AttributesBuilder in the future, who knows)
* Deprecate default BaseTracer constructor: library instrumentations are supposed to inject all dependencies, the javaagent should explicitly pass globals
* fix compilation failure
* Remove peer.service customization from library instrumentation
* Make BaseTracer fields private
By making `tracer` private we're forcing all tracer implementation to use the `spanBuilder()` utility method and pass the parent context manually.
* MethodsConfigurationParser is moved to javaagent-tooling - it's only used in some instrumentation modules
* BootstrapPackagePrefixesHolder is moved to javaagent-api - it just needs to be in the bootstrap CL for the classloader instrumentation, so it does not necessarily need to be instrumentation-api