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

MethodJmsListenerEndpoint does not set the DestinationResolver on the MessagingMessageListenerAdapter [SPR-12927] #17520

Closed
spring-issuemaster opened this issue Apr 17, 2015 · 3 comments

Comments

Projects
None yet
2 participants
@spring-issuemaster
Copy link
Collaborator

commented Apr 17, 2015

Stephan Oudmaijer opened SPR-12927 and commented

The MethodJmsListenerEndpoint does not set a destination resolver in the createMessageListener() method. Therefore it always uses the DynamicDestinationResolver defined at spring-jms-4.1.6.RELEASE-sources.jar!/org/springframework/jms/listener/adapter/AbstractAdaptableMessageListener.java:60


Affects: 4.1.6

Issue Links:

  • #17651 MessagingMessageListenerAdapter should use DestinationResolver of MessageListenerContainer ("is duplicated by")

Referenced from: commits bd05e3a, 444b903

@spring-issuemaster

This comment has been minimized.

Copy link
Collaborator Author

commented Apr 20, 2015

Stéphane Nicoll commented

This is fixed and only for the @JmsListener infrastructure. The XML-based config had the same problem (probably since the beginning) and I guess that users have created a custom message listener adapter solution in this case. I am not too keen to change the default at this point.

@spring-issuemaster

This comment has been minimized.

Copy link
Collaborator Author

commented May 22, 2015

Juergen Hoeller commented

Marking this for 4.1.7 as well, as per the discussion on #17651.

@spring-issuemaster

This comment has been minimized.

Copy link
Collaborator Author

commented May 22, 2015

Stéphane Nicoll commented

This has been backported to 4.1

Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
You can’t perform that action at this time.