Mateusz Rzeszutek
95ec4a8c1f
HTTP semconv: filter out default peer/host ports ( #7258 )
...
After a loooong break, the next part of the HTTP semconv implementation:
filtering out default HTTP ports, client and server.
> [5]: If not default (80 for http scheme, 443 for https).
https://github.com/open-telemetry/opentelemetry-specification/blob/main/specification/trace/semantic_conventions/http.md#http-client
2022-11-22 17:14:20 +00:00
Mateusz Rzeszutek
72cc791b39
Remove deprecated HTTP semconv code ( #7259 )
2022-11-21 11:30:40 -08:00
jason plumb
d4b29cf521
Migrate some SQL sanitizer tests to java ( #7148 )
2022-11-15 06:15:32 +00:00
Trask Stalnaker
01c1f249ba
Add deprecation cycle for removed methods ( #7020 )
...
Methods were removed in #6892 , which was backported to the 1.19.x branch
and included in the 1.19.1 release.
2022-11-01 12:25:28 -07:00
Aaron Ai
3911cacfdc
Fix typos ( #6961 )
2022-10-24 09:06:01 -07:00
Trask Stalnaker
94e2248940
Use new semantic attribute constants ( #6946 )
2022-10-23 09:32:03 -07:00
Trask Stalnaker
9ce68fb937
Fix metric units ( #6931 )
...
Follow-up to #6930
2022-10-22 10:00:48 -07:00
Trask Stalnaker
f488d94403
Capture net.host.name for netty ( #6892 )
...
This may be a regression in 1.19.0 because you can no longer reconstruct
the original url for netty server spans (previously `http.host` was
captured which could be used).
2022-10-19 09:21:50 -07:00
jack-berg
aeac361816
Upgrade to otel java 1.19.0 ( #6757 )
...
Working PR to capture all the changes required to update to otel java
1.19.0. The new log API force allows
`:instrumentation-appender-api-internal` and
`:instrumentation-appender-sdk-internal`, but necessitates a decent
amount of refactoring as a result.
The PR points at the `1.19.0-SNAPSHOT`, which I'll update upon
publication.
Co-authored-by: Mateusz Rzeszutek <mrzeszutek@splunk.com>
Co-authored-by: Trask Stalnaker <trask.stalnaker@gmail.com>
Co-authored-by: Lauri Tulmin <ltulmin@splunk.com>
2022-10-12 09:19:37 -07:00
Mateusz Rzeszutek
77035fc88c
Extract `net.peer.{name,port}` on start for CLIENT spans ( #6828 )
...
The [HTTP
spec](https://github.com/open-telemetry/opentelemetry-specification/blob/main/specification/trace/semantic_conventions/http.md#http-client )
says these two attributes must be provided at span creation time - I
think it makes sense to extend it over to all `net`-related
instrumentations, cause these are supposed to be the logical peer
name/port, which are supposed to be known before the connection is
started/exchange is made.
2022-10-10 16:00:19 -07:00
Malte
7117f4226d
ISSUE-2674 Strip sensitive data from the url ( #6417 )
...
Fixes #2674 by replacing basic auth information as part of the URL with
`username:password`.
Co-authored-by: Malte <MALPI@users.noreply.github.com>
Co-authored-by: Mateusz Rzeszutek <mrzeszutek@splunk.com>
Co-authored-by: Trask Stalnaker <trask.stalnaker@gmail.com>
2022-10-07 13:18:23 +02:00
Mateusz Rzeszutek
714ba9189e
Remove deprecated `http.host` and `http.server_name` attributes ( #6709 )
...
... and make sure the `TemporaryMetricsView` follows the current spec
2022-09-24 11:02:21 -07:00
Trask Stalnaker
429ecfc713
Update error prone ( #6646 )
...
(note that change from BDDMockito is due to
https://github.com/google/error-prone/issues/3396 )
2022-09-23 11:24:40 -07:00
Trask Stalnaker
d5c6453636
Remove external usages of ClassNames (and move it to internal) ( #6584 )
...
* Reduce dependency on simpleName
* Move ClassNames to internal
* Javadoc
* Updates
* Fix
* Spotless
* Rename
* Add test
2022-09-13 13:58:19 -07:00
Mateusz Rzeszutek
f50f1fed3b
Fix instrumentation-api-semconv packages ( #6590 )
...
* Fix instrumentation-api-semconv packages
* Move test
Co-authored-by: Trask Stalnaker <trask.stalnaker@gmail.com>
2022-09-12 14:51:27 -07:00
Mateusz Rzeszutek
cfdbe758f2
Net attributes getters changes (in preparation for HTTP spec impl) ( #6503 )
...
* Net attributes getters changes: instrumentation-api-semconv changes
* Net attributes getters changes: getter implementations
* Net attributes getters changes: test fixes
* Remove net.sock.host.name
* code review comments
* default getter methods & getPeerSocketAddress() method name
* set authority in grpc earlier
2022-09-12 09:20:27 -07:00
Trask Stalnaker
63c0bfbf79
Add grpc status code to metrics attrs ( #6556 )
2022-09-08 12:48:09 +03:00
jason plumb
0b5a22c685
Migrate HTTP StatusConverter tests from groovy to java ( #6537 )
...
* migrate HttpServerStatusConverterTest from groovy to java
* migrate HttpServerClientConverterTest from groovy to java
* rename
* convert DynamicTest to ParameterizedTest.
2022-09-02 11:52:09 +02:00
Mateusz Rzeszutek
59cb9ca0df
Remove deprecated code ( #6501 )
...
* Remove deprecated code
* unnecessary semicolon
* fix distro and extension examples
2022-08-24 14:36:57 -07:00
Trask Stalnaker
f1774cabe2
Update net semantic convention changes ( #6268 )
...
* New net conventions: option a
* Feedback + sock.family + sock.peer.name
* peer.service + tests
* server net attributes attempt 1
* server net attributes attempt 2
* Javadoc
* Revisions
* Apply to instrumentations
* Feedback
* One more default method
* Spotless
* Fix javadoc
2022-08-18 09:02:23 -07:00
Mateusz Rzeszutek
6cd79b2a94
Deprecate the 2-arg variant of HttpCommonAttributesGeter#statusCode() ( #6466 )
...
* Deprecate the 2-arg variant of HttpCommonAttributesGeter#statusCode()
* fix liberty
2022-08-12 17:11:11 -07:00
Lauri Tulmin
07d7cfd551
Capture messaging header value as span attribute ( #6454 )
...
* Capture messaging header value as span attribute
* add comment
2022-08-11 17:28:04 -07:00
Trask Stalnaker
68a9f20eb3
Enable PrivateConstructorForUtilityClass errorprone check ( #6427 )
...
* PrivateConstructorForUtilityClass
* Advice
* More advice
* More
* More advice
* More
* Spotless
* Fix
* Fix
* Fix
* A better solution
* Revert
* More
* Fix
* Spotless
* Fix
2022-08-10 11:30:22 +03:00
Mateusz Rzeszutek
0d1cc8ea0a
Set http.route in spring-autoconfigure webmvc instrumentation ( #6414 )
...
* Set http.route in spring-autoconfigure webmvc instrumentation
* Bump spring-webmvc library instrumentation version to 5.3
* nit: protected -> private
* Remove duplicated test (already covered by HttpSpanStatusExtractorTest)
* Move the README to the correct module
* fix link
* fix more links
* liiiiiiinks
* fix tests
* remove not needed weakref
2022-08-09 09:36:28 -07:00
Mateusz Rzeszutek
b2c90c79b5
Extract HTTP request & response content length from headers ( #6415 )
...
* Extract HTTP request & response content length from headers
* remove unused method
* fix camel tests
* fix google http client tests
* fix HttpUrlConnection tests
* fix k8s and jaxrs tests
* fix aws tests
* actually fix aws tests 🤞
* fix elasticsearch tests
* fix ratpack tests
* fix spring webflux tests
* fix vertx tests
* fix reactor netty tests
2022-08-05 10:55:47 -07:00
Mateusz Rzeszutek
5a152cd289
Deprecate uncompressed content length HTTP attributes ( #6383 )
2022-07-29 06:17:38 +02:00
jack-berg
c518bf8d7e
Add http.client|server.request|response.size metrics ( #6376 )
2022-07-27 09:28:11 -07:00
Mateusz Rzeszutek
6a24080fe9
Add code attributes to several instrumentations ( #6365 )
...
* Add code attributes to several instrumentations
* fix servlet tests
* fix tests
* fix elasticsearch tests
* fix context bridge test
* fix rxjava tests
* fix spring webmvc tests
2022-07-27 09:37:01 +02:00
Mateusz Rzeszutek
976ab9411c
Deprecate Config ( #6360 )
...
* Deprecate Config
* suppress deprecation on ConfigTest
2022-07-25 12:10:11 -07:00
Mateusz Rzeszutek
82b39b1012
Rename `newInstrumenter()` into `buildInstrumenter()` ( #6363 )
...
* Rename newInstrumenter() into buildInstrumenter()
* spotless
2022-07-25 12:02:46 -07:00
Mateusz Rzeszutek
8d1ba17d29
InstrumentationConfig part 4: DB sanitization ( #6317 )
...
* InstrumentationConfig part 4: DB sanitization
* fix couchbase unit tests
* code review comments
Co-authored-by: Trask Stalnaker <trask.stalnaker@gmail.com>
2022-07-15 21:47:33 -07:00
Mateusz Rzeszutek
3af56e7d22
InstrumentationConfig part 3: HTTP headers and peer service mappings ( #6302 )
2022-07-15 14:52:52 -07:00
Mateusz Rzeszutek
7bbe918008
InstrumentationConfig part 2 ( #6292 )
2022-07-11 09:47:19 +02:00
Mateusz Rzeszutek
e7887ac929
Idea: deprecate `Config`, add agent-only `InstrumentationConfig` ( #6264 )
...
* Idea: deprecate Config, add agent-only InstrumentationConfig
* fix
* fix camel test
* fix external-annotation tests
2022-07-08 16:20:49 +02:00
Mateusz Rzeszutek
afee828846
Use metrics batch API in connection pool instrumentations ( #6231 )
...
* Use metrics batch API in connection pool instrumentations
* Fix vibur muzzle
* Fix vibur muzzle
2022-07-05 13:47:15 +02:00
Mateusz Rzeszutek
72e0b26b2b
Always pass Context when recording HttpServerMetrics ( #6223 )
2022-06-27 10:23:36 -07:00
HaloFour
b7fc80c98e
Change SpanStatusExtractor to use a builder that can set status description ( #6035 )
...
* Change SpanStatusExtractor to use a builder that can set status descripion
* Refactor SpanStatusExtractor to only support builder approach to setting status
* Revert setting the exception as the status description
* PR feedback
* Re-fix graghql instrumentation span extractor
* Spotless
2022-05-19 10:00:46 -07:00
Mateusz Rzeszutek
5bcab32379
Implement HikariCP connection pool metrics ( #6003 )
...
* Implement HikariCP connection pool metrics
* rebase after SDK update
* fix muzzle
* code review comments
2022-05-12 18:48:07 -07:00
Mateusz Rzeszutek
87b412fb40
Instrumentation API changes: OperationMetrics, OperationListener ( #6016 )
...
* Instrumentation API changes: OperationMetrics, OperationListener
* errorprone
2022-05-12 11:16:46 -07:00
Anuraag Agrawal
d1ee692479
Update to OTel 1.14 ( #5999 )
...
* Update to OTel 1.14
* Fix test
Co-authored-by: Trask Stalnaker <trask.stalnaker@gmail.com>
2022-05-10 14:23:56 -07:00
Mateusz Rzeszutek
802f5aa3ea
Instrumentation API and Javadoc cleanup ( #5954 )
2022-05-09 22:13:45 -07:00
Trask Stalnaker
e3588ceee3
Fix some jul messages ( #5930 )
2022-04-27 10:33:12 -07:00
Mateusz Rzeszutek
2bb7873f99
Add javaagent<->application context bridge for HttpRouteHolder ( #5838 )
...
* Add javaagent<->application context bridge for HttpRouteHolder
* remove comments
* fix broken http.route bridge
* spotless
* Move to a separate module
2022-04-22 08:29:54 -07:00
Mateusz Rzeszutek
2c4acd7c81
Introduce LocalRootSpan (replacing ServerSpan) ( #5896 )
...
* Introduce LocalRootSpan (replacing ServerSpan)
* fix tests
* bridge old ServerSpan class
2022-04-21 11:29:10 +02:00
Mateusz Rzeszutek
4e3f19d469
Enable span suppression by SpanKey by default ( #5779 )
...
* Enable span suppression by SpanKey by default
* fix HTTP tests (probably)
* add exception for camel
* remove suppression tests from @WithSpan instrumentations
* remove suppression tests from @WithSpan instrumentation; spring boot autoconfigure
* fix twilio tests
* fix netty-based HTTP clients, remove AWS SDK 1.11 unit test
* fix elasticsearch tests
* codenarc
* spotless
* fix AWS SDK 1.11 tests
* remove a TODO
* code review comments
* fix merge conflict
Co-authored-by: Trask Stalnaker <trask.stalnaker@gmail.com>
2022-04-19 14:13:09 -07:00
Mateusz Rzeszutek
492c022dd5
Fix duplicate class error on Android build ( #5882 )
2022-04-19 12:47:41 -07:00
Mateusz Rzeszutek
202a24c867
Remove deprecated code from instrumentation-api, extension-api ( #5883 )
2022-04-19 12:22:11 -07:00
Mateusz Rzeszutek
1ad0342ab0
Update the http.route attribute even for not sampled server spans ( #5844 )
2022-04-15 10:15:49 -07:00
Mateusz Rzeszutek
e9fbb367b3
Rename netty `always-create-connect-span` property to `connection-tel… ( #5834 )
...
* Rename netty `always-create-connect-span` property to `connection-telemetry`
* formatting
* warn when deprecated property is used
* errorprone
2022-04-15 13:38:04 +02:00
Anuraag Agrawal
e58d39d4ad
Update to OTel 1.13 ( #5799 )
...
* Update to OTel 1.13
* Fix test
* Foo
* Fix app server test and start yak shaving
* Yak
* Yak
* groovy fail
* Yak
* GROOVY
* yak farm
2022-04-13 13:16:32 +03:00