opentelemetry-java-instrume.../testing-common
Trask Stalnaker 05471b053b
Webflux instrumentation fix (#7251)
When a webflux filter is added which throws an exception, the
instrumentation does not currently capture the `http.status_code`.

The fix is to move `WebClientTracingFilter` from the first to the last
filter in the chain, which I think(?) is the general strategy we've
taken for other client instrumentation, e.g. so that if a filter makes
another http call it won't be suppressed.

I don't love the test coverage I added, so let me know if you have any
better suggestions?

EDIT: btw, I did archaeology to confirm that behavior (adding to the
beginning of the chain) has been in place since the webflux
instrumentation was added originally
6f472a62a0 (diff-493ad89b5bde807c90387aa2bb67eb10d3bcef6b6a388bd31e11796a6d01ac38R36)
2022-11-22 15:09:22 -08:00
..
integration-tests End groovy files with newline (#7127) 2022-11-10 12:21:55 +01:00
library-for-integration-tests Fix a bug in the field backed VirtualField implementation (#4310) 2021-10-06 09:40:53 -07:00
src Webflux instrumentation fix (#7251) 2022-11-22 15:09:22 -08:00
build.gradle.kts Don't bundle slf4j-api inside testing-commons (#6520) 2022-08-30 00:04:26 +03:00