Skip to content

Instrument RestClient for Observability #31114

New issue

Have a question about this project? Sign up for a free GitHub account to open an issue and contact its maintainers and the community.

Sign up for GitHub

By clicking “Sign up for GitHub”, you agree to our terms of service and privacy statement. We’ll occasionally send you account related emails.

Already on GitHub? Sign in to your account

Jump to bottom
Closed
bclozel opened this issue Aug 25, 2023 · 0 comments
Closed

Instrument RestClient for Observability #31114

bclozel opened this issue Aug 25, 2023 · 0 comments
Assignees
@bclozel
Labels
in: web Issues in web modules (web, webmvc, webflux, websocket) theme: observability An issue related to observability and tracing type: enhancement A general enhancement
Milestone
6.1.0-M5

Comments

@bclozel
Copy link
Member

bclozel commented Aug 25, 2023

RestClient has been recently introduced and we should instrument it for observability.
This client variant uses the same infrastructure as the RestTemplate, so we should share the observation context and conventions. The client style is more functional, so the instrumentation itself will be similar to WebClient.
@bclozel bclozel added in: web Issues in web modules (web, webmvc, webflux, websocket) type: enhancement A general enhancement theme: observability An issue related to observability and tracing labels Aug 25, 2023
@bclozel bclozel added this to the 6.1.0-RC1 milestone Aug 25, 2023
@bclozel bclozel self-assigned this Aug 25, 2023
@hpoettker hpoettker mentioned this issue Sep 27, 2023
Closed
@bclozel bclozel mentioned this issue Feb 22, 2025
Closed
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Assignees

@bclozel bclozel

Labels
in: web Issues in web modules (web, webmvc, webflux, websocket) theme: observability An issue related to observability and tracing type: enhancement A general enhancement
Projects
None yet
Milestone
6.1.0-M5
Development

No branches or pull requests
1 participant
@bclozel