Fix: set user on transaction

[maciejwalkowiak]

📜 Description

Set user on transaction.

User is set on the scope in Servlet filter - SentryUserFilter. This filter must run after Spring security (or any other filter verifying authentication details) so that SentryUserProvider implementations can access the currently logged in user data.

In Spring Boot integration, SentryUserFilter is set to run:

• by default runs as the last filter (lowest precedence)
• user can explicitly define the order

In Spring integration, filter bean must be created manually and a corresponding DelegatingFilterProxy servlet filter must be registered.

💡 Motivation and Context

Fixes #1335

💚 How did you test it?

📝 Checklist

• I reviewed the submitted code
• I added tests to verify the changes
• I updated the docs if needed
• No breaking changes

[codecov-commenter]

Codecov Report

Merging #1443 (376a316) into main (a33afef) will decrease coverage by 0.01%.
The diff coverage is 70.83%.

@@             Coverage Diff              @@
##               main    #1443      +/-   ##
============================================
- Coverage     75.88%   75.87%   -0.02%     
- Complexity     1889     1901      +12     
============================================
  Files           187      189       +2     
  Lines          6453     6499      +46     
  Branches        633      640       +7     
============================================
+ Hits           4897     4931      +34     
- Misses         1261     1272      +11     
- Partials        295      296       +1     

Impacted Files	Coverage Δ	Complexity Δ
.../io/sentry/spring/SentryInitBeanPostProcessor.java	97.22% <ø> (ø)	6.00 <0.00> (ø)
...entry/spring/SentryUserProviderEventProcessor.java	96.55% <ø> (ø)	12.00 <0.00> (ø)
...entry/spring/SpringSecuritySentryUserProvider.java	0.00% <0.00%> (ø)	0.00 <0.00> (?)
...n/java/io/sentry/spring/boot/SentryProperties.java	76.66% <33.33%> (-4.82%)	9.00 <1.00> (+1.00)	⬇️
...c/main/java/io/sentry/spring/SentryUserFilter.java	92.85% <92.85%> (ø)	10.00 <10.00> (?)
...io/sentry/spring/boot/SentryAutoConfiguration.java	98.18% <100.00%> (+0.18%)	1.00 <0.00> (ø)
.../io/sentry/spring/tracing/SentryTracingFilter.java	81.57% <0.00%> (+5.26%)	7.00% <0.00%> (+1.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 a33afef...376a316. Read the comment docs.

[bruno-garcia]

Looks like we need to add these to the docs and a note on a migration plan for the next version (should be 5.0 I believe)?

[marandaneto]

@maciejwalkowiak we need docs for this
getsentry/sentry-docs#3482

[marandaneto]

LGTM