Skip to content
New issue

Have a question about this project? Sign up for a free GitHub account to open an issue and contact its maintainers and the community.

By clicking “Sign up for GitHub”, you agree to our terms of service and privacy statement. We’ll occasionally send you account related emails.

Already on GitHub? Sign in to your account

Support spring web 6 / boot 3 #4605

Merged
merged 14 commits into from Feb 2, 2023
Merged

Support spring web 6 / boot 3 #4605

merged 14 commits into from Feb 2, 2023

Conversation

amarziali
Copy link
Collaborator

What Does This Do

Add support for spring web 6. Old instrumentation was not compatible because of jakarta EE 9 package changes.
Some side notes:

  • Testing required a big refactoring effort to have test app compatible with spring boot 3
  • I added some FIXME on tests. Some stuff are usually addressed by servlet instrumentation (that's missing because we do not yet support servlet 5). They are minor misses

Motivation

Additional Notes

@amarziali amarziali requested a review from a team as a code owner January 26, 2023 15:38
@smola smola added the inst: others All other instrumentations label Jan 27, 2023
@devinsba devinsba requested a review from a team January 30, 2023 14:53
Copy link
Contributor

@devinsba devinsba left a comment

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

It looks good to me. AppSec should verify their classes look good too

@smola smola requested a review from cataphract January 31, 2023 09:27
@amarziali amarziali added the inst: spring Spring instrumentation label Jan 31, 2023
@amarziali amarziali added the tag: do not merge Do not merge changes label Feb 1, 2023
@amarziali amarziali force-pushed the andrea.marziali/springweb6 branch 2 times, most recently from a7aa4ff to d508c76 Compare February 1, 2023 10:40
@amarziali amarziali removed the tag: do not merge Do not merge changes label Feb 1, 2023
@amarziali amarziali merged commit d67bb4c into master Feb 2, 2023
@amarziali amarziali deleted the andrea.marziali/springweb6 branch February 2, 2023 14:45
@github-actions github-actions bot added this to the 1.7.0 milestone Feb 2, 2023
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
inst: others All other instrumentations inst: spring Spring instrumentation
Projects
None yet
Development

Successfully merging this pull request may close these issues.

None yet

5 participants