Skip to content

Fix deserializing lists with @SnsNotificationMessage #1282

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

Merged
merged 1 commit into from
Dec 11, 2024

Conversation

frjonsen
Copy link
Contributor

📢 Type of change

  • Bugfix
  • New feature
  • Enhancement
  • Refactoring

📜 Description

When deserialization target is a list, get the inner type

💡 Motivation and Context

#1280

💚 How did you test it?

Modifying the existing test to access the actual fields of the deserialized objects

📝 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

@github-actions github-actions bot added the component: sqs SQS integration related issue label Nov 24, 2024
@maciejwalkowiak maciejwalkowiak added the type: bug Something isn't working label Dec 11, 2024
@maciejwalkowiak maciejwalkowiak added this to the 3.3.0 M2 milestone Dec 11, 2024
@maciejwalkowiak
Copy link
Contributor

Thanks @frjonsen!

@maciejwalkowiak maciejwalkowiak merged commit a2da6d2 into awspring:main Dec 11, 2024
4 checks passed
@maciejwalkowiak maciejwalkowiak changed the title Properly deserialize lists with @SnsNotificationMessage Fix deserializing lists with @SnsNotificationMessage Dec 11, 2024
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: bug Something isn't working
Projects
None yet
Development

Successfully merging this pull request may close these issues.

2 participants