Skip to content

Fix Observation Filter docs#38570

Closed
aleksandrserbin wants to merge 1 commit into
spring-projects:mainfrom
aleksandrserbin:patch-1
Closed

Fix Observation Filter docs#38570
aleksandrserbin wants to merge 1 commit into
spring-projects:mainfrom
aleksandrserbin:patch-1

Conversation

@aleksandrserbin

Copy link
Copy Markdown
Contributor

WebMvcMetricsFilter no longer exists since 3.0 and should be replaced with ServerHttpObservationFilter. Documentation still references the old class name

`WebMvcMetricsFilter` no longer exists since 3.0 and should be replaced with `ServerHttpObservationFilter`
@pivotal-cla

Copy link
Copy Markdown

@aleksandrserbin 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 label Nov 27, 2023
@pivotal-cla

Copy link
Copy Markdown

@aleksandrserbin Thank you for signing the Contributor License Agreement!

@mhalbritter mhalbritter added type: documentation A documentation update and removed status: waiting-for-triage An issue we've not yet triaged labels Nov 28, 2023
@mhalbritter mhalbritter added this to the 3.1.x milestone Nov 28, 2023
@mhalbritter mhalbritter self-assigned this Nov 28, 2023
@mhalbritter

Copy link
Copy Markdown
Contributor

Thank you very much and congratulations on your first contribution 🎉!

@mhalbritter mhalbritter modified the milestones: 3.1.x, 3.1.7 Nov 28, 2023
mhalbritter pushed a commit that referenced this pull request Nov 28, 2023
`WebMvcMetricsFilter` no longer exists since 3.0 and should be
replaced with `ServerHttpObservationFilter`

See gh-38570
@aleksandrserbin aleksandrserbin deleted the patch-1 branch November 30, 2023 11:11
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Labels

type: documentation A documentation update

Projects

None yet

Development

Successfully merging this pull request may close these issues.

4 participants