-
-
Couldn't load subscription status.
- Fork 460
Hubs/Scopes Merge 11 - Replace IHub with IScopes in Spring 5 / Spring Boot 2 integrations
#3308
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
Hubs/Scopes Merge 11 - Replace IHub with IScopes in Spring 5 / Spring Boot 2 integrations
#3308
Conversation
|
Performance metrics 🚀
|
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
Comment on class EnableSentry refer to Hub instead of Scopes.
Also, should we rename the SentryHubRegistrar and its methods?
sentry-spring/src/main/java/io/sentry/spring/webflux/SentryWebFilter.java
Show resolved
Hide resolved
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
Should we rename HubConfiguration in SentryAutoConfiguration?
|
|
||
| @Bean | ||
| public @NotNull IHub sentryHub( | ||
| public @NotNull IScopes sentryHub( |
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
Should we rename this method?
|
We need to test whether renaming Spring beans and config classes breaks anything. Even then user code could break by renaming them. |
#skip-changelog
📜 Description
Follow up for #3297 changing Spring 5 / Spring Boot 2 integrations.
💡 Motivation and Context
💚 How did you test it?
📝 Checklist
sendDefaultPIIis enabled.🔮 Next steps