Add new modules for Spring 7 and Spring Boot 4 #4601

adinauer · 2025-08-04T14:32:26Z

📜 Description

Copy the existing SDK and sample modules over to new ones for Spring 7 and Spring Boot 4.
Change build.gradle.kts files to use the new versions of Spring/Boot in the newly created modules.
Change package and SDK name in build.gradle.kts files.
Update integration tests, ignored modules, .craft.yml (commented out), GH actions and issue templates to use the new modules as necessary.

💡 Motivation and Context

Initial work for #4080

💚 How did you test it?

📝 Checklist

I added tests to verify the changes.
No new PII added or SDK only sends newly added PII if sendDefaultPII is enabled.
I updated the docs if needed.
I updated the wizard if needed.
Review from the native team if needed.
No breaking change or entry added to the changelog.
No breaking change for hybrid SDKs or communicated to hybrid SDKs.

🔮 Next steps

adinauer · 2025-08-04T14:32:50Z

This stack of pull requests is managed by Graphite. Learn more about stacking.

github-actions · 2025-08-04T14:45:33Z

Performance metrics 🚀

	Plain	With Sentry	Diff
Startup time	421.54 ms	522.34 ms	100.80 ms
Size	1.58 MiB	2.10 MiB	533.45 KiB

Baseline results on branch: main

Startup times

Revision	Plain	With Sentry	Diff
`ee747ae`	396.82 ms	441.67 ms	44.86 ms
`ee747ae`	382.73 ms	435.41 ms	52.68 ms
`ee747ae`	386.94 ms	431.43 ms	44.49 ms
`ee747ae`	357.79 ms	421.84 ms	64.05 ms
`ee747ae`	554.98 ms	611.50 ms	56.52 ms
`ee747ae`	358.21 ms	389.41 ms	31.20 ms
`674d437`	355.28 ms	504.18 ms	148.90 ms
`ee747ae`	400.46 ms	423.61 ms	23.15 ms
`ee747ae`	374.71 ms	455.18 ms	80.47 ms
`ee747ae`	415.92 ms	470.15 ms	54.23 ms

App size

Revision	Plain	With Sentry	Diff
`ee747ae`	1.58 MiB	2.10 MiB	530.95 KiB
`ee747ae`	1.58 MiB	2.10 MiB	530.95 KiB
`ee747ae`	1.58 MiB	2.10 MiB	530.95 KiB
`ee747ae`	1.58 MiB	2.10 MiB	530.95 KiB
`ee747ae`	1.58 MiB	2.10 MiB	530.95 KiB
`ee747ae`	1.58 MiB	2.10 MiB	530.95 KiB
`674d437`	1.58 MiB	2.10 MiB	530.94 KiB
`ee747ae`	1.58 MiB	2.10 MiB	530.95 KiB
`ee747ae`	1.58 MiB	2.10 MiB	530.95 KiB
`ee747ae`	1.58 MiB	2.10 MiB	530.95 KiB

Previous results on branch: 08-04-add_new_modules_for_spring_7_and_spring_boot_4

Startup times

Revision	Plain	With Sentry	Diff
`596a7b4`	381.65 ms	424.59 ms	42.95 ms
`0deda87`	378.14 ms	394.35 ms	16.20 ms
`c6da6b0`	431.38 ms	494.80 ms	63.42 ms
`0fef848`	426.67 ms	455.08 ms	28.41 ms
`b969500`	441.45 ms	478.17 ms	36.72 ms
`4be2375`	449.48 ms	480.81 ms	31.33 ms
`e7c4664`	416.04 ms	441.27 ms	25.22 ms
`b2c2d0b`	414.00 ms	447.42 ms	33.42 ms
`3b8514c`	403.45 ms	468.29 ms	64.84 ms
`d2060c4`	432.98 ms	460.16 ms	27.18 ms

App size

Revision	Plain	With Sentry	Diff
`596a7b4`	1.58 MiB	2.10 MiB	533.28 KiB
`0deda87`	1.58 MiB	2.10 MiB	533.46 KiB
`c6da6b0`	1.58 MiB	2.10 MiB	533.45 KiB
`0fef848`	1.58 MiB	2.09 MiB	522.33 KiB
`b969500`	1.58 MiB	2.10 MiB	533.46 KiB
`4be2375`	1.58 MiB	2.10 MiB	533.45 KiB
`e7c4664`	1.58 MiB	2.09 MiB	521.83 KiB
`b2c2d0b`	1.58 MiB	2.09 MiB	521.57 KiB
`3b8514c`	1.58 MiB	2.10 MiB	533.45 KiB
`d2060c4`	1.58 MiB	2.10 MiB	533.25 KiB

lcian

I've reviewed the changes in the root directory plus the diffs between each of the old and new modules, and everything looks correct to me.

…g_boot_4

alexander-alderman-webb · 2025-08-08T11:03:59Z

.github/workflows/system-tests-backend.yml

+          - sample: "sentry-samples-spring-boot-jakarta"
+            agent: "false"
+            agent-auto-init: "true"


is there a reason this sample was not added before to the system-tests-backend (since the sample is not added by this PR)?

Good catch, this shouldn't be here. It's a duplicate since it's already there:

matrix: sample: [ "sentry-samples-spring-boot-jakarta" ] agent: [ "false" ] agent-auto-init: [ "true" ]

* Fix Spring 7 and Spring Boot 4 modules * Attempt to fix kotlin 2.2 issue * Second attempt to fix Kotlin 2.2 (#4610) * Second attempt to fix Kotlin 2.2 * Format code * Third attempt at fixing Kotlin 2.2 compat (#4613) --------- Co-authored-by: Sentry Github Bot <[email protected]> * Address compose-related issues * Fix tests for Spring 7 and Spring Boot 4 (#4614) * Second attempt to fix Kotlin 2.2 * Format code * Third attempt at fixing Kotlin 2.2 compat * Fix tests for Spring 7 and Spring Boot 4 * Move Spring 7 and Spring Boot 4 packages (#4615) * Move Spring 7 and Spring Boot 4 packages * Fix class not found due to OTel not supporting spring boot 4 yet (#4616) * Format code * some fixes * change kotlin 1.8 to 1.9 and some cleanup * ignore warnings about api level that is not relevant * fix optional dependencies in SentryAutoConfiguration * Update trace origin * Remove duplicate e2e test config * Update Strings for Spring 7 and Spring Boot 4 * Disable Spring Boot 4 agentless e2e tests for now --------- Co-authored-by: Sentry Github Bot <[email protected]> --------- Co-authored-by: Sentry Github Bot <[email protected]> Co-authored-by: markushi <[email protected]>

…g_boot_4

cursor · 2025-08-26T09:31:28Z

sentry-spring-7/src/main/java/io/sentry/spring7/graphql/SentryGraphqlConfiguration.java

+  public SentryInstrumentation sentryInstrumentationWebMvc(
+      final @NotNull ObjectProvider<SentryGraphqlInstrumentation.BeforeSpanCallback>
+              beforeSpanCallback) {
+    SentryIntegrationPackageStorage.getInstance().addIntegration("Spring7GrahQLWebMVC");


Bug: GraphQL Typo in Spring Integration Names

There's a consistent typo in the GraphQL integration names "Spring7GrahQLWebMVC" and "Spring7GrahQLWebFlux" across the new configurations. The "GrahQL" spelling should be "GraphQL", which impacts how these integrations are reported and tracked.

Additional Locations (4)

sentry-spring-boot-4/src/main/java/io/sentry/spring/boot4/graphql/SentryGraphql22AutoConfiguration.java#L30-L31

sentry-spring-boot-4/src/main/java/io/sentry/spring/boot4/graphql/SentryGraphqlAutoConfiguration.java#L30-L31

sentry-spring-boot-4/src/main/java/io/sentry/spring/boot4/graphql/SentryGraphql22AutoConfiguration.java#L41-L42

sentry-spring-7/src/main/java/io/sentry/spring7/graphql/SentryGraphql22Configuration.java#L30-L31

This was referenced Aug 4, 2025

Fix Spring 7 and Spring Boot 4 modules #4602

Merged

Attempt to fix kotlin 2.2 issue #4603

Merged

adinauer force-pushed the 08-04-add_new_modules_for_spring_7_and_spring_boot_4 branch from 9899937 to e7a97e5 Compare August 6, 2025 14:34

adinauer mentioned this pull request Aug 6, 2025

Second attempt to fix Kotlin 2.2 #4610

Merged

7 tasks

Add new modules for Spring 7 and Spring Boot 4

3ec789b

adinauer force-pushed the 08-04-add_new_modules_for_spring_7_and_spring_boot_4 branch from e7a97e5 to 3ec789b Compare August 7, 2025 13:02

This was referenced Aug 7, 2025

Third attempt at fixing Kotlin 2.2 compat #4613

Merged

Fix tests for Spring 7 and Spring Boot 4 #4614

Merged

Move Spring 7 and Spring Boot 4 packages #4615

Merged

Fix class not found due to OTel not supporting spring boot 4 yet #4616

Merged

lcian approved these changes Aug 8, 2025

View reviewed changes

adinauer added 2 commits August 8, 2025 12:02

Fix Spring 7 and Spring Boot 4 modules (#4602)

ddeada2

Merge branch 'main' into 08-04-add_new_modules_for_spring_7_and_sprin…

1b6c6e0

…g_boot_4

adinauer marked this pull request as ready for review August 8, 2025 10:02

adinauer requested review from romtsn, stefanosiano and markushi as code owners August 8, 2025 10:02

This comment was marked as outdated.

Sign in to view

alexander-alderman-webb reviewed Aug 8, 2025

View reviewed changes

adinauer and others added 2 commits August 11, 2025 14:12

Merge branch 'main' into 08-04-add_new_modules_for_spring_7_and_sprin…

a18e719

…g_boot_4

This comment was marked as outdated.

Sign in to view

adinauer and others added 2 commits August 11, 2025 14:23

changelog

975f6c7

Merge branch 'main' into 08-04-add_new_modules_for_spring_7_and_sprin…

47b3629

…g_boot_4

This comment was marked as outdated.

Sign in to view

adinauer and others added 3 commits August 13, 2025 15:44

Merge branch 'main' into 08-04-add_new_modules_for_spring_7_and_sprin…

36e68d4

…g_boot_4

fix ci

1ae3d92

add ignored span origins for Spring 7 and Spring Boot 4

fe36a4e

adinauer and others added 3 commits August 14, 2025 11:18

move changelog

d3164a3

Merge branch 'main' into 08-04-add_new_modules_for_spring_7_and_sprin…

ff7c269

…g_boot_4

Merge branch 'main' into 08-04-add_new_modules_for_spring_7_and_sprin…

9634d01

…g_boot_4

This comment was marked as outdated.

Sign in to view

alexander-alderman-webb and others added 3 commits August 18, 2025 08:30

Example how to fix version mismatch (#4642)

65ed9c6

Fix ignored span origin

a1438d0

Merge branch 'main' into 08-04-add_new_modules_for_spring_7_and_sprin…

ac1b238

…g_boot_4

This comment was marked as outdated.

Sign in to view

Merge branch 'main' into 08-04-add_new_modules_for_spring_7_and_sprin…

41946bf

…g_boot_4

This comment was marked as outdated.

Sign in to view

move changelog

5535263

cursor bot reviewed Aug 26, 2025

View reviewed changes

adinauer merged commit 3699cd5 into main Aug 26, 2025
46 of 47 checks passed

adinauer deleted the 08-04-add_new_modules_for_spring_7_and_spring_boot_4 branch August 26, 2025 10:34

Uh oh!

Add new modules for Spring 7 and Spring Boot 4 #4601

Add new modules for Spring 7 and Spring Boot 4 #4601

Uh oh!

Conversation

adinauer commented Aug 4, 2025 • edited Loading Uh oh! There was an error while loading. Please reload this page.

Uh oh!

📜 Description

💡 Motivation and Context

💚 How did you test it?

📝 Checklist

🔮 Next steps

Uh oh!

adinauer commented Aug 4, 2025 • edited Loading Uh oh! There was an error while loading. Please reload this page.

Uh oh!

Uh oh!

github-actions bot commented Aug 4, 2025 • edited Loading Uh oh! There was an error while loading. Please reload this page.

Uh oh!

Performance metrics 🚀

Baseline results on branch: main

Startup times

App size

Previous results on branch: 08-04-add_new_modules_for_spring_7_and_spring_boot_4

Startup times

App size

Uh oh!

lcian left a comment • edited Loading Uh oh! There was an error while loading. Please reload this page.

Uh oh!

Choose a reason for hiding this comment

Uh oh!

This comment was marked as outdated.

Uh oh!

alexander-alderman-webb Aug 8, 2025

Choose a reason for hiding this comment

Uh oh!

adinauer Aug 11, 2025

Choose a reason for hiding this comment

Uh oh!

This comment was marked as outdated.

Uh oh!

This comment was marked as outdated.

Uh oh!

This comment was marked as outdated.

Uh oh!

This comment was marked as outdated.

Uh oh!

This comment was marked as outdated.

Uh oh!

cursor bot Aug 26, 2025

Choose a reason for hiding this comment

Bug: GraphQL Typo in Spring Integration Names

Uh oh!

Uh oh!

Uh oh!

adinauer commented Aug 4, 2025 •

edited

Loading

adinauer commented Aug 4, 2025 •

edited

Loading

github-actions bot commented Aug 4, 2025 •

edited

Loading

lcian left a comment •

edited

Loading