sroze/messenger-enqueue-transport

Default receive timeout not compatible with SQS

AlexStansfield opened this issue · 1 comments

Just to let you know that the default receive timeout of 30 seconds introduced in version 0.2.0:

if (null === ($interopMessage = $this->getConsumer()->receive($this->options['receiveTimeout'] ?? 30000))) {

Is incompatible with SQS (max 20 seconds):

https://github.com/php-enqueue/sqs/blob/b203de327b4f5045717b9420de071a19c0f78afc/SqsConsumer.php#L88

Not really a big deal but wasn't sure if 30 seconds was picked for a specific or arbitrary reason.

We're fixing our setup by setting the receive timeout, which is probably the best solution anyway, but thought you might consider updating the 30 seconds to 20 just to avoid confusion.

Have the same issue. Watching where the 30s come from.
Thanks a lot.

Is it possible to change the default value to 20s please?