Skip to content

Removes SLF4JBridgeHandler from opentelemetry module - #4043

Open
froque wants to merge 1 commit into
jooby-project:mainfrom
froque:4042-remove-SLF4JBridgeHandler
Open

froque wants to merge 1 commit into
jooby-project:mainfrom
froque:4042-remove-SLF4JBridgeHandler

Conversation

@froque

@froque froque commented Nov 10, 2026 •

Copy link
Copy Markdown
Contributor

Breaks with stackoverflow when final application uses slf4j-jdk14 logging implementation.

Fixes #4042

Breaks with stackoverflow when final application uses slf4j-jdk14 logging
implementation.
@codecov

codecov Bot commented Nov 10, 2026

Copy link
Copy Markdown

Codecov Report

✅ All modified and coverable lines are covered by tests.
✅ Project coverage is 94.19%. Comparing base (89370b8) to head (eccac0c).

Additional details and impacted files
@@             Coverage Diff              @@
##               main    #4043      +/-   ##
============================================
- Coverage     94.22%   94.19%   -0.03%     
+ Complexity     6694     6690       -4     
============================================
  Files           432      432              
  Lines         20964    20962       -2     
  Branches       2108     2108              
============================================
- Hits          19754    19746       -8     
- Misses          757      763       +6     
  Partials        453      453              

☔ View full report in Codecov by Harness.
📢 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.

@jknack jknack added this to the 4.6.1 milestone Nov 10, 2026

This branch has not been deployed

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

Projects

None yet

Development

Successfully merging this pull request may close these issues.

jooby-opentelemetry not compatible with slf4j-jdk14

2 participants