Skip to content

Conversation

@vadimi
Copy link

@vadimi vadimi commented Jan 26, 2026

This version performs the same instrumentation as previously available otelecho in opentelemetry-go-contrib repo, but with echo v5 support

v5 sub-directory is used specifically in case of new v6, v7, etc versions of echo come out in the future

This version performs the same instrumentation as previously available
`otelecho` in `opentelemetry-go-contrib` repo, but with echo v5 support

v5 sub-directory is used specifically in case of new v6, v7, etc
versions of echo come out in the future
@aldas aldas self-assigned this Jan 26, 2026
@codecov
Copy link

codecov bot commented Jan 26, 2026

Codecov Report

✅ All modified and coverable lines are covered by tests.
✅ Project coverage is 67.69%. Comparing base (1e69e54) to head (f3a0cc8).
⚠️ Report is 8 commits behind head on master.

Additional details and impacted files
@@            Coverage Diff             @@
##           master     #141      +/-   ##
==========================================
+ Coverage   63.77%   67.69%   +3.91%     
==========================================
  Files           9        9              
  Lines         969      585     -384     
==========================================
- Hits          618      396     -222     
+ Misses        308      157     -151     
+ Partials       43       32      -11     

☔ View full report in Codecov by Sentry.
📢 Have feedback on the report? Share it here.

🚀 New features to boost your workflow:
  • ❄️ Test Analytics: Detect flaky tests, report on failures, and find test suite problems.

@aldas
Copy link
Contributor

aldas commented Jan 29, 2026

Just a quick heads-up that I will be reviewing this over the weekend.

@mojixcoder
Copy link

mojixcoder commented Jan 31, 2026

Thanks, this PR is heavily needed and is our only blocker to upgrade to v5.
Moving this package to echo-contrib has a little difference with the otelecho one which is addressed in #143 .
But I think it's out of scope of this PR.

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.

3 participants