Skip to content

Add Helidon Instrumentation #13776

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.

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

Draft
wants to merge 7 commits into
base: main
Choose a base branch
from

Conversation

SentryMan
Copy link
Contributor

@SentryMan SentryMan commented Apr 25, 2025

Adds helidon support

Resolves #11809 and helidon-io/helidon#10051

@tjquinno
Copy link

@SentryMan Your PR in Helidon that fixed the related problem is in release 4.2.3 which was released last week.

IIRC that was the blocker for this PR. Might you be able to try this one using 4.2.3 and see if it works?

@SentryMan
Copy link
Contributor Author

@SentryMan Your PR in Helidon that fixed the related problem is in release 4.2.3 which was released last week.

There is no matchingPattern method on ServerRequest in this version

@tjquinno
Copy link

tjquinno commented Jun 12, 2025

Ugh. You're right. I based my previous note on just a quick glance at the PR date vs. the release date of 4.2.3, but your bug fix was for an enhancement PR, and we are (almost) never including enhancements in dot-dot releases.

So 4.3 is the right release to use.

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.

Add Support for the Helidon Webserver
2 participants