Thanks to visit codestin.com
Credit goes to github.com

Skip to content

Clarify documentation for the default uri client request observation metric#34116

Closed
Mattias-Sehlstedt wants to merge 1 commit into
spring-projects:mainfrom
Mattias-Sehlstedt:docs-uri-observability-metric
Closed

Clarify documentation for the default uri client request observation metric#34116
Mattias-Sehlstedt wants to merge 1 commit into
spring-projects:mainfrom
Mattias-Sehlstedt:docs-uri-observability-metric

Conversation

@Mattias-Sehlstedt

Copy link
Copy Markdown
Contributor

Change the description for the uri client request observation metric to clarify that only the protocol, host and port parts are removed from the template with the default observability convention.

Adds tests to illustrate that it is intentional that the query parameters are kept as part of the template.

closes #34107

@pivotal-cla

Copy link
Copy Markdown

@Mattias-Sehlstedt Please sign the Contributor License Agreement!

Click here to manually synchronize the status of this Pull Request.

See the FAQ for frequently asked questions.

@spring-projects-issues spring-projects-issues added the status: waiting-for-triage An issue we've not yet triaged or decided on label Dec 18, 2024
@bclozel bclozel self-assigned this Dec 18, 2024
@bclozel bclozel added type: documentation A documentation task in: web Issues in web modules (web, webmvc, webflux, websocket) theme: observability and removed status: waiting-for-triage An issue we've not yet triaged or decided on labels Dec 18, 2024
@bclozel bclozel modified the milestone: 6.2.2 Dec 18, 2024
@bclozel bclozel added the status: waiting-for-triage An issue we've not yet triaged or decided on label Dec 18, 2024
@bclozel

bclozel commented Dec 18, 2024

Copy link
Copy Markdown
Member

Thanks for your PR! Just realized that you didn't sign the CLA yet @Mattias-Sehlstedt, I'll put this issue back in triage until this is done.

@bclozel bclozel added the status: waiting-for-feedback We need additional information before we can continue label Dec 18, 2024
@Mattias-Sehlstedt

Copy link
Copy Markdown
Contributor Author

I keep getting "Reauthorization required" when trying to sign the agreement, and I cannot see it as a know issue on the pivotal page (https://cla.pivotal.io/about). I have tried revoking the access and then enabling it again but the issue persists. Could this be considered an "Obvious fix", or should I be in contact with Pivotal and ask them about it?

image

@spring-projects-issues spring-projects-issues added status: feedback-provided Feedback has been provided and removed status: waiting-for-feedback We need additional information before we can continue labels Dec 19, 2024
@bclozel

bclozel commented Dec 19, 2024

Copy link
Copy Markdown
Member

Thanks for trying; I'll see what I can do.
In the meantime, if your contribution matches what the https://developercertificate.org/ says, you can force push your commit and signoff on it with git commit --signoff.

…to describe what parts that are removed from the template

Signed-off-by: Mattias-Sehlstedt <[email protected]>
@Mattias-Sehlstedt Mattias-Sehlstedt force-pushed the docs-uri-observability-metric branch from 07bc906 to 7ea938b Compare December 19, 2024 11:10
@pivotal-cla

Copy link
Copy Markdown

@Mattias-Sehlstedt Thank you for signing the Contributor License Agreement!

@bclozel bclozel removed status: waiting-for-triage An issue we've not yet triaged or decided on status: feedback-provided Feedback has been provided labels Jan 7, 2025
@bclozel bclozel added this to the 6.2.2 milestone Jan 7, 2025
@bclozel bclozel closed this in 50b1fb0 Jan 7, 2025
@bclozel

bclozel commented Jan 7, 2025

Copy link
Copy Markdown
Member

Thanks @Mattias-Sehlstedt for your contribution!
FYI we're moving away from the CLA and using signoffs now so this should avoid such problems in the future.

@Mattias-Sehlstedt Mattias-Sehlstedt deleted the docs-uri-observability-metric branch January 7, 2025 11:37
@Mattias-Sehlstedt

Copy link
Copy Markdown
Contributor Author

Thanks for the quick feedback and assistance @bclozel, makes it a very pleasant experience to contribute to open source πŸ‘

Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Labels

in: web Issues in web modules (web, webmvc, webflux, websocket) type: documentation A documentation task

Projects

None yet

Development

Successfully merging this pull request may close these issues.

Unclear documentation for the uri observability metric in HTTP clients

4 participants