feat: bump sqs producer and consumer which uses aws sdk v3 under the hood#9152
Open
kevin9foong wants to merge 1 commit intodevelopfrom
Open
feat: bump sqs producer and consumer which uses aws sdk v3 under the hood#9152kevin9foong wants to merge 1 commit intodevelopfrom
kevin9foong wants to merge 1 commit intodevelopfrom
Conversation
kevin9foong
changed the title
This was referenced Feb 26, 2026
kevin9foong
commented
Feb 26, 2026
| export const createWebhookQueueHandler = | ||
| (producer: WebhookProducer) => | ||
| async (sqsMessage: aws.SQS.Message): Promise<void> => { | ||
| async (sqsMessage: Message): Promise<Message | undefined> => { |
Contributor
Author
There was a problem hiding this comment.
Note: Instead of returning resolved promise to remove the message from queue and acknowledge, the new sqs-consumer expects the original message to be returned.
kevin9foong
commented
Feb 26, 2026
| queueUrl, | ||
| region: config.aws.region, | ||
| handleMessage: createWebhookQueueHandler(producer), | ||
| // By default, the default Node.js HTTP/HTTPS SQS agent |
Contributor
Author
There was a problem hiding this comment.
note: can be safely removed, since the new sqs-consumer uses AWS SDK v3 under the hood, which HTTP keep-alive is on by default.
This comment has been minimized.
This comment has been minimized.
This file contains hidden or bidirectional Unicode text that may be interpreted or compiled differently than what appears below. To review, open the file in an editor that reveals hidden Unicode characters.
Learn more about bidirectional Unicode characters
Sign up for free
to join this conversation on GitHub.
Already have an account?
Sign in to comment
1 participant
Add this suggestion to a batch that can be applied as a single commit.This suggestion is invalid because no changes were made to the code.Suggestions cannot be applied while the pull request is closed.Suggestions cannot be applied while viewing a subset of changes.Only one suggestion per line can be applied in a batch.Add this suggestion to a batch that can be applied as a single commit.Applying suggestions on deleted lines is not supported.You must change the existing code in this line in order to create a valid suggestion.Outdated suggestions cannot be applied.This suggestion has been applied or marked resolved.Suggestions cannot be applied from pending reviews.Suggestions cannot be applied on multi-line comments.Suggestions cannot be applied while the pull request is queued to merge.Suggestion cannot be applied right now. Please check back later.
Problem
Closes dep bumps for sqs-consumer and sqs-producer
Note that the updates use the new AWS SDK v3 from the previous AWS SDK v2 which is EOL since Sept 2025.
Solution
Since keep alive is now default enabled for AWS SDK v3, we can safely remove the sqs argument which aims to keep the SQS session alive previously recommended in v5.7.0 of sqs-consumer.
Breaking Changes
No - this PR is backwards compatible
Tests
Webhook can be received:
Webhook with retries can be received: