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

[cds-756] Add support for DLQ #81

Closed
wants to merge 37 commits into from
Closed

[cds-756] Add support for DLQ #81

wants to merge 37 commits into from

Conversation

daidokoro
Copy link
Contributor

This PR adds support for enabling a DLQ

  • added functionality to handle nested sqs events using receursion
  • add sqs config values
  • modify parameters to handler function to use new clients type
  • updated tests to use the new clients type
  • added new dependencies

Description

Fixes #

How Has This Been Tested?

Checklist:

  • I have updated the versions in the SemanticVersion in template.yaml
  • I have updated the CHANGELOG.md
  • I have created necessary PR to Terraform Module Repository (https://github.com/coralogix/terraform-coralogix-aws) if needed
  • This change does not affect any particular component (e.g. it's CI or docs change)

added s3 upload function for failed sqs retry limits
updated dlq s3 event workflow to store original event if object is unavailable
…arameter

based function and added functionality to the custom resource to allow
it to configure lambda for DLQ
@daidokoro daidokoro closed this Apr 18, 2024
@daidokoro daidokoro deleted the cds-756 branch April 18, 2024 14:56
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
None yet
Projects
None yet
Development

Successfully merging this pull request may close these issues.

1 participant