Add support for async dispatch requests

[aesy]

Keeps the transaction open until the response is committed.

Fixes #3982

[adinauer]

Hey, sorry for the long wait here. I just took a quick look and the way this is implemented at the moment would break distributed tracing. I'll update the PR to fix this.

Also this is still based on v7 main, I'll update to current main.

[adinauer]

This change would break existing functionality. We would no longer be parsing and propagating incoming tracing information or create a new trace in case there's no incoming info. This would break distributed tracing, which allows you to track execution from e.g. an App across multiple servers.

[adinauer]

I've updated the PR.

• Merged in latest version of main
• Fixed the tracing without performance problem
• Added a config option to enable this behaviour via sentry.keep-transactions-open-for-async-responses=true for now. We can revisit at a later point whether this should become opt-out.
• Tried adding some comments / naming method so we know how this works when coming back to the code in the future.

I'll let someone else on the team review this, since I've changed quite a bit.

You may also want to give our OpenTelemetry based SDK a try if you can't wait for this change to be released. Docs on how to setup can be found here: https://docs.sentry.io/platforms/java/guides/spring-boot/

[lcian]

Ooops didn't mean to approve but just to comment.
Anyway, could we add tests? @adinauer
Maybe one of the comments I highlighted above is wrong and the booleans are confusing me, but it's not immediately clear to me that this works as intended.

[lcian]

Looks good!

[adinauer]

Thanks again for the PR @aesy, we've just released this as part of https://github.com/getsentry/sentry-java/releases/tag/8.3.0

[aesy]

That's great, thank you for the update!