Skip to content

Introduce @EnableMessagingAnnotations to activate messaging annotations post processors#11029

Closed
mjd507 wants to merge 1 commit into
spring-projects:mainfrom
mjd507:gh-11024-option2
Closed

Introduce @EnableMessagingAnnotations to activate messaging annotations post processors#11029
mjd507 wants to merge 1 commit into
spring-projects:mainfrom
mjd507:gh-11024-option2

Conversation

@mjd507
Copy link
Copy Markdown
Contributor

@mjd507 mjd507 commented May 29, 2026

this is another option for #11024

…ions post processors

this is another option for spring-projects#11024

Signed-off-by: Jiandong Ma <jiandong.ma.cn@gmail.com>
new IntegrationRegistrar()
.registerBeanDefinitions(importingClassMetadata, registry);
new MessagingAnnotationsRegister()
.registerBeanDefinitions(importingClassMetadata, registry);
Copy link
Copy Markdown
Contributor Author

Choose a reason for hiding this comment

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

activate messaging annotations for ClassPathXmlApplicationContext that uses <integration:annotation-config/>

@Documented
@Import(IntegrationRegistrar.class)
@Import({IntegrationRegistrar.class})
@EnableMessagingAnnotations
Copy link
Copy Markdown
Contributor Author

Choose a reason for hiding this comment

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

activate by default, currently not a good time to fix tests.

@mjd507
Copy link
Copy Markdown
Contributor Author

mjd507 commented May 29, 2026

there is a @IntegrationComponentScan used for scanning MessagingGateway, SpringBoot by default auto-configured it. looks it will out of this @EnableMessagingAnnotations control. what we want is not to scan if user not enable messaging annotations. just raise this concern.

@artembilan
Copy link
Copy Markdown
Member

Interesting idea, and it really was proposed on the other PR.
However, this is still not what we would like to go right now.
Need more engineering thinking.
So, let's continue discussion in the issue!

Thanks

Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Labels

None yet

Projects

None yet

Development

Successfully merging this pull request may close these issues.

2 participants