Support logging via JUL

[maciejwalkowiak]

📜 Description

Support logging via JUL + sample project.

💡 Motivation and Context

Fixes #1037

💚 How did you test it?

Unit & Integration tests.

📝 Checklist

• I reviewed the submitted code
• I added tests to verify the changes
• No breaking changes

🔮 Next steps

Create a PR in Quarkus or let know Quarkus team about JUL logger in Sentry SDK, so that they do not need to maintain their custom one.

[maciejwalkowiak]

Afterthought - for the backward compatibility we should also support MDC tags, which means we need a dependency for SLF4J, which means that JUL appender would have to go to separate module.

[codecov-io]

Codecov Report

Merging #1211 (3390bd2) into main (d58b440) will decrease coverage by 0.05%.
The diff coverage is 71.90%.

@@             Coverage Diff              @@
##               main    #1211      +/-   ##
============================================
- Coverage     74.62%   74.57%   -0.06%     
- Complexity     1682     1713      +31     
============================================
  Files           176      177       +1     
  Lines          5908     6029     +121     
  Branches        578      600      +22     
============================================
+ Hits           4409     4496      +87     
- Misses         1234     1256      +22     
- Partials        265      277      +12     

Impacted Files	Coverage Δ	Complexity Δ
...jul/src/main/java/io/sentry/jul/SentryHandler.java	71.90% <71.90%> (ø)	31.00 <31.00> (?)

---

Continue to review full report at Codecov.

Legend - Click here to learn more
Δ = absolute <relative> (impact), ø = not affected, ? = missing data
Powered by Codecov. Last update d58b440...3390bd2. Read the comment docs.

[bruno-garcia]

Afterthought - for the backward compatibility we should also support MDC tags, which means we need a dependency for SLF4J, which means that JUL appender would have to go to separate module.

Sorry but I don't follow. before it wasnt' a separate package, so why does it need to be now for backwards compat?

[marandaneto]

Afterthought - for the backward compatibility we should also support MDC tags, which means we need a dependency for SLF4J, which means that JUL appender would have to go to separate module.

Sorry but I don't follow. before it wasnt' a separate package, so why does it need to be now for backwards compat?

adding a dependency of SLF4J I believe.

[maciejwalkowiak]

As @marandaneto said, since 3.x we do not have slf4j dependency in the SDK core. I am not really sure how people use slf4j AND jul at the same time, but since it was in 1.x and it's also in handler added to Quarkus, I think we should follow the same path.

[bruno-garcia]

makes sense

[marandaneto]

a few comments but looks good and ready to go.
LGTM
The next step would be to create docs too, probably linking to the migration page.

[marandaneto]

@maciejwalkowiak CI is not happy