Repository navigation
Cap queue delays on Laravel Cloud's SQS-backed driver - #1074
Merged
Merged
Conversation
Laravel Cloud's managed queues register a `cloud` driver that wraps SQS, so timers and v2 task dispatches longer than 900 seconds failed with SQS InvalidParameterValue. Treat `cloud` like `sqs` everywhere the 900s dispatch cap and SQS release cap apply. Reported and first fixed for 1.x by Joel-Jensen in #1072. Co-Authored-By: Claude Opus 5.5 <noreply@anthropic.com>
|
rmcdaniel
commented
Nov 11, 2026
rmcdaniel
left a comment
rmcdaniel
left a comment
Member
Author
There was a problem hiding this comment.
Reviewed. BackendCapabilities::isSqsBackedDriver() now covers all three 'sqs' checks on main: the v1 Timers cap, the v2 max_delay_seconds capability and the TimerTransportChunker release cap. No other SQS driver checks remain in src. New tests cover the cloud dispatch (900s) and release (43200s) caps. All checks pass on c28e17d. Merging.
This was referenced Nov 11, 2026
rsguimaraes-pe
pushed a commit
to rsguimaraes-pe/workflow
that referenced
this pull request
Nov 13, 2026
coverage-unit runs the unit suite only after merge, so a unit test that fails there could merge green. durable-workflow#1074's Laravel Cloud timer test did exactly that. A new pr-unit job runs the same suite without coverage, with the same SQLite and Redis setup, and the build gate requires it on pull requests. Co-Authored-By: Claude Opus 5.5 <noreply@anthropic.com>
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
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.
Laravel Cloud's managed queues register a
cloudqueue driver that wraps SQS. Workflow only applied the 900-second SQS delay cap when the driver was literallysqs, so on Laravel Cloud any timer,awaitWithTimeout()or delayed v2 task over 15 minutes failed withDelaySeconds must be >= 0 and <= 900.This carries #1072 (1.x, by @Joel-Jensen) forward to
main:BackendCapabilities::isSqsBackedDriver()names the SQS-backed drivers (sqs,cloud) in one place.max_delay_secondscapability, the v2 release cap inTimerTransportChunker, and the v1Timers::timer()cap.ECS and PHPStan pass on the changed files locally; unit/feature suites run in CI.
🤖 Generated with Claude Code