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.

Sign up for GitHub

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

Jump to bottom

Add MessagingMessageConverter to SQS Auto-configuration #1145

Closed
tomazfernandes opened this issue May 2, 2024 · 1 comment
Closed

Add MessagingMessageConverter to SQS Auto-configuration #1145

tomazfernandes opened this issue May 2, 2024 · 1 comment
Labels
component: sqs SQS integration related issue status: ideal-for-contribution We agree it's nice to have but it is not team priority type: enhancement Smaller enhancement in existing integration

Comments

@tomazfernandes
Copy link
Contributor

Currently the only way to set a custom MessagingMessageConverter is to declare a SqsMessageListenerContainerFactory and set it manually, which generates friction since when a factory bean is declared the user no longer benefits from its auto configuration.

We should make it possible to declare a MessagingMessageConverter bean and having it set in auto configuration.
@tomazfernandes tomazfernandes added component: sqs SQS integration related issue type: enhancement Smaller enhancement in existing integration status: ideal-for-contribution We agree it's nice to have but it is not team priority labels May 2, 2024
@imsosleepy imsosleepy mentioned this issue May 12, 2024
Closed
9 tasks
@injae-kim
Copy link

PR: #1150

Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Assignees
No one assigned
Labels
component: sqs SQS integration related issue status: ideal-for-contribution We agree it's nice to have but it is not team priority type: enhancement Smaller enhancement in existing integration
Projects
None yet
Milestone
No milestone
Development

No branches or pull requests
2 participants
@injae-kim @tomazfernandes