Skip to content

fix: pcntl signal reaching Go runtime causing long running cli scripts to get stuck#2445

Open
henderkes wants to merge 3 commits into
mainfrom
fix/long-running-phpcli
Open

fix: pcntl signal reaching Go runtime causing long running cli scripts to get stuck#2445
henderkes wants to merge 3 commits into
mainfrom
fix/long-running-phpcli

Conversation

@henderkes
Copy link
Copy Markdown
Contributor

@henderkes henderkes commented May 24, 2026

fixes #1902
fixes #2326
probably fixes #871
closes #614

edit: draft until I can manually verify with a real laravel/symfony reproducer

@henderkes henderkes marked this pull request as draft May 24, 2026 15:16
@henderkes henderkes marked this pull request as ready for review May 25, 2026 13:45
@henderkes
Copy link
Copy Markdown
Contributor Author

Clanker got this one right: docker run --rm -t dubbleclick/frankenphp-1902-repro

https://github.com/laravel/framework/blob/v13.11.2/src/Illuminate/Queue/Worker.php#L853-L880
https://github.com/laravel/horizon/blob/v5.47.0/src/WorkerProcess.php#L67-L141

Workers set up signal receivers and the orchestrator issues them: https://github.com/laravel/horizon/blob/v5.47.0/src/Supervisor.php#L187-L220

@henderkes henderkes changed the title fix: signal handlers reaching Go runtime causing long running cli scripts to get stuck fix: pcntl signal reaching Go runtime causing long running cli scripts to get stuck May 25, 2026
@henderkes henderkes requested a review from withinboredom May 25, 2026 14:40
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Labels

None yet

Projects

None yet

1 participant