Skip to content

Add autoconfiguration for AcknowledgementResultCallback components#1584

Open
rviniciussouza wants to merge 4 commits intoawspring:mainfrom
rviniciussouza:main
Open

Add autoconfiguration for AcknowledgementResultCallback components#1584
rviniciussouza wants to merge 4 commits intoawspring:mainfrom
rviniciussouza:main

Conversation

@rviniciussouza
Copy link

📢 Type of change

  • Bugfix
  • New feature
  • Enhancement
  • Refactoring

📜 Description

Add auto-configuration for AcknowledgementResultCallback and AsyncAcknowledgementResultCallback in the default SQS listener container factory.

💡 Motivation and Context

Only AsyncAcknowledgementResultCallback and AcknowledgementResultCallback currently require manual wiring in the default SqsListenerContainerFactory.

This PR removes that requirement by automatically attaching them when present in the application context.

Issue: #1583

💚 How did you test it?

  • Unit tests: Added tests in SqsAutoConfigurationTest and LegacySqsAutoConfigurationTest to cover scenarios where the callbacks are either declared or absent.

📝 Checklist

  • I reviewed submitted code
  • I added tests to verify changes
  • I updated reference documentation to reflect the change
  • All tests passing
  • No breaking changes

🔮 Next steps

@github-actions github-actions bot added the component: sqs SQS integration related issue label Feb 22, 2026
Copy link
Contributor

@tomazfernandes tomazfernandes left a comment

Choose a reason for hiding this comment

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

Thanks for the PR @rviniciussouza.

Can you add an example in the docs under the Acknowledgement Result Callback section?

You can use the ErrorHandler section as reference.

Thanks.

@github-actions github-actions bot added the type: documentation Documentation or Samples related issue label Feb 28, 2026
Copy link
Contributor

@tomazfernandes tomazfernandes left a comment

Choose a reason for hiding this comment

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

@rviniciussouza looks good, thanks!

#1562 introduced a conflict in the docs, you might need to do some reordering so the flow makes sense.

Also, please add "Since 4.1.0" for the new behavior.

@rviniciussouza
Copy link
Author

Hi, @tomazfernandes .
To resolve the conflict in sqs.adoc, I moved the content added in #1562 to make the text cleaner.

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

Labels

component: sqs SQS integration related issue type: documentation Documentation or Samples related issue

Projects

None yet

Development

Successfully merging this pull request may close these issues.

3 participants