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

SQS receiveMessage makes 2 requests #1582

Closed
zamu87 opened this issue Oct 19, 2020 · 5 comments
Closed

SQS receiveMessage makes 2 requests #1582

zamu87 opened this issue Oct 19, 2020 · 5 comments
Assignees
Labels
needs-triage This issue or PR still needs to be triaged.

Comments

@zamu87
Copy link

zamu87 commented Oct 19, 2020

Describe the bug
Using SQS receiveMessage in a FIFO queue, 2 identical requests are generated causing a long delay in returning result because the second request wait for VisibilityTimeout. Swapping the SDK to v2 resolves the issue.

SDK version number
1.0.0-gamma.11

Is the issue in the browser/Node.js/ReactNative?
Node.js

Details of the browser/Node.js/ReactNative version
v14.14.0

To Reproduce (observed behavior)

const params = {
            AttributeNames: ["SentTimestamp"],
            MaxNumberOfMessages: 1, 
            MessageAttributeNames: ["All"],
            QueueUrl: queueUrl,
            VisibilityTimeout: 20,
            WaitTimeSeconds: 20,
        };
const response = await this.connection.receiveMessage(params);

Expected behavior
A single request should be done.

@alexforsyth alexforsyth self-assigned this Oct 30, 2020
@alexforsyth alexforsyth added the needs-triage This issue or PR still needs to be triaged. label Oct 30, 2020
@ShogunPanda
Copy link
Contributor

I can confirm this also happens in non FIFO queues.

@artur-ma
Copy link

Can confirm also

@rahulswarnkar
Copy link

Something similar on send side #1742?

@trivikr trivikr assigned trivikr and unassigned alexforsyth Dec 7, 2020
@trivikr
Copy link
Member

trivikr commented Dec 8, 2020

The issue with receiveMessage was fixed in rc.6 in PR #1680
The fix for sendMessage is posted in #1742, and will be published with rc.9

@trivikr trivikr closed this as completed Dec 8, 2020
@github-actions
Copy link

github-actions bot commented Jan 8, 2021

This thread has been automatically locked since there has not been any recent activity after it was closed. Please open a new issue for related bugs and link to relevant comments in this thread.

@github-actions github-actions bot locked as resolved and limited conversation to collaborators Jan 8, 2021
Sign up for free to subscribe to this conversation on GitHub. Already have an account? Sign in.
Labels
needs-triage This issue or PR still needs to be triaged.
Projects
None yet
Development

No branches or pull requests

6 participants