Skip to content

Move experimental servlet method out of public API#15826

Merged
trask merged 1 commit intoopen-telemetry:mainfrom
trask:servlet-experimental
Jan 12, 2026
Merged

Move experimental servlet method out of public API#15826
trask merged 1 commit intoopen-telemetry:mainfrom
trask:servlet-experimental

Conversation

@trask
Copy link
Copy Markdown
Member

@trask trask commented Jan 10, 2026

Moving to experimental since there's no corresponding semantic convention for this (unlike unknown methods for example).

Related to

@trask trask force-pushed the servlet-experimental branch from 19b374c to 079341e Compare January 10, 2026 21:04
@trask trask marked this pull request as ready for review January 10, 2026 21:40
@trask trask requested a review from a team as a code owner January 10, 2026 21:40
}
}

public static void setCapturedRequestParameters(
Copy link
Copy Markdown
Contributor

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

I think we might need to add javadoc to the experimental methods

Copy link
Copy Markdown
Member Author

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

I added this as TODO on #12846, will send follow-up

@trask trask merged commit 7d62d59 into open-telemetry:main Jan 12, 2026
85 checks passed
@trask trask deleted the servlet-experimental branch January 12, 2026 16:58
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Labels

None yet

Projects

None yet

Development

Successfully merging this pull request may close these issues.

2 participants