Skip to content

feat(event_sources,parser): add support for DeadLetterQueueSourceArn for SQS #4133

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 2 commits into from
Apr 15, 2024

Conversation

rubenfonseca
Copy link
Contributor

Issue number: #4111

Summary

Changes

Please provide a summary of what's being changed

This PR adds support for a new DeadLetterQueueSourceArn field on the SQS record data class and parser.

The field is present when a Lambda function uses a SQS DLQ as an event source.

User experience

Please share what the user experience looks like before and after this change

After this change you could access the field when receiving records from a SQS DLQ.

Checklist

If your change doesn't seem to apply, please leave them unchecked.

Is this a breaking change?

RFC issue number:

Checklist:

  • Migration process documented
  • Implement warnings (if it can live side by side)

Acknowledgment

By submitting this pull request, I confirm that you can use, modify, copy, and redistribute this contribution, under the terms of your choice.

Disclaimer: We value your time and bandwidth. As such, any pull requests created on non-triaged issues might not be successful.

@boring-cyborg boring-cyborg bot added the tests label Apr 15, 2024
@pull-request-size pull-request-size bot added the size/M Denotes a PR that changes 30-99 lines, ignoring generated files. label Apr 15, 2024
@github-actions github-actions bot added the feature New feature or functionality label Apr 15, 2024
@rubenfonseca rubenfonseca marked this pull request as ready for review April 15, 2024 13:38
@rubenfonseca rubenfonseca requested a review from a team April 15, 2024 13:38
@rubenfonseca rubenfonseca added event_sources Event Source Data Class utility parser Parser (Pydantic) utility labels Apr 15, 2024
Copy link
Contributor

@leandrodamascena leandrodamascena left a comment

Choose a reason for hiding this comment

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

Thanks @rubenfonseca. Just 2 small changes.

Co-authored-by: Leandro Damascena <[email protected]>
Signed-off-by: Ruben Fonseca <[email protected]>
Copy link

Quality Gate Passed Quality Gate passed

Issues
0 New issues
0 Accepted issues

Measures
0 Security Hotspots
No data about Coverage
0.0% Duplication on New Code

See analysis details on SonarCloud

@rubenfonseca rubenfonseca merged commit 16e2e7d into develop Apr 15, 2024
14 checks passed
@rubenfonseca rubenfonseca deleted the rf/4111 branch April 15, 2024 15:10
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
event_sources Event Source Data Class utility feature New feature or functionality parser Parser (Pydantic) utility size/M Denotes a PR that changes 30-99 lines, ignoring generated files. tests
Projects
None yet
Development

Successfully merging this pull request may close these issues.

Static typing: Missing DeadLetterQueueSourceArn on SqsAttributesModel
2 participants