gnodet commented on PR #26395: URL: https://github.com/apache/camel/pull/26395#issuecomment-5663146564
Thanks @apupier for the deep-dive request — you were right to push on this. **Analysis of the Spring Boot situation:** The Spring Boot `SqsDelayedQueueTest` uses the same buggy `defaultVisibilityTimeout=0` in `BaseSqs.receiveMessageFromQueue`. The test has been annotated `@DisabledIfSystemProperty(named = "ci.env.name", matches = "github.com")` since December 2023 (commit `94102a5`), so it has **never run on CI** in its current form and there is no evidence it ever passed with `defaultVisibilityTimeout=0` against a modern LocalStack. The two parameters are semantically distinct: - `defaultVisibilityTimeout` → maps to `SetQueueAttributes(VISIBILITY_TIMEOUT)`, called during `Sqs2Endpoint.doInit()` when the endpoint connects to an existing queue. On LocalStack, this call resets `DELAY_SECONDS` to 0. - `visibilityTimeout` → maps to `ReceiveMessage(VisibilityTimeout)` (per-request), does **not** call `SetQueueAttributes`, does not touch `DELAY_SECONDS`. **Title fix:** Updated to "Add SqsProducerDelayedQueueIT" as requested. **Spring Boot fix:** Opened https://github.com/apache/camel-spring-boot/pull/1976 which applies the same `defaultVisibilityTimeout→visibilityTimeout` fix to `BaseSqs.receiveMessageFromQueue`. -- This is an automated message from the Apache Git Service. To respond to the message, please log on to GitHub and use the URL above to go to the specific comment. To unsubscribe, e-mail: [email protected] For queries about this service, please contact Infrastructure at: [email protected]
