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

DefaultJmsListenerContainerFactoryConfigurer#setObservationRegistry should not be public #44466

Closed
vpavic opened this issue Feb 27, 2025 · 2 comments
Assignees
Labels
type: bug A general bug
Milestone

Comments

@vpavic
Copy link
Contributor

vpavic commented Feb 27, 2025

While working on #44465 I noticed that DefaultJmsListenerContainerFactoryConfigurer#setObservationRegistry (introduced in 0321a8a) is public unlike other setters in this class that are package-private. There are no usages of #setObservationRegistry outside of its package so this was likely by accident.

@spring-projects-issues spring-projects-issues added the status: waiting-for-triage An issue we've not yet triaged label Feb 27, 2025
@snicoll snicoll added the for: team-attention An issue we'd like other members of the team to review label Mar 4, 2025
@snicoll
Copy link
Member

snicoll commented Mar 4, 2025

Thanks for the report. While there isn't any usage of it, it is now public in a GA version of Spring Boot so I am not sure if we could change that in a maintenance release. Flagging for team attention to see what the rest of the team think.

@wilkinsona wilkinsona changed the title Review visibility of DefaultJmsListenerContainerFactoryConfigurer#setObservationRegistry Deprecate DefaultJmsListenerContainerFactoryConfigurer#setObservationRegistry as it should not be public Mar 5, 2025
@wilkinsona
Copy link
Member

We're going to deprecate setObservationRegistry in 3.3.x onwards as it should not have been public. Once the deprecation period has elapsed, we'll make it package private. In the meantime, callers elsewhere in Boot can suppress the warning.

@wilkinsona wilkinsona changed the title Deprecate DefaultJmsListenerContainerFactoryConfigurer#setObservationRegistry as it should not be public DefaultJmsListenerContainerFactoryConfigurer#setObservationRegistry should not be public Mar 5, 2025
@wilkinsona wilkinsona added type: bug A general bug and removed for: team-attention An issue we'd like other members of the team to review status: waiting-for-triage An issue we've not yet triaged labels Mar 5, 2025
@wilkinsona wilkinsona added this to the 3.3.x milestone Mar 5, 2025
@snicoll snicoll self-assigned this Mar 5, 2025
@snicoll snicoll modified the milestones: 3.3.x, 3.3.10 Mar 6, 2025
@snicoll snicoll closed this as completed in 6cf81e2 Mar 6, 2025
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
type: bug A general bug
Projects
None yet
Development

No branches or pull requests

4 participants