You signed in with another tab or window. Reload to refresh your session.You signed out in another tab or window. Reload to refresh your session.You switched accounts on another tab or window. Reload to refresh your session.Dismiss alert
I'm using Ubuntu. Today, when trying to run "symfony serve" I got a message "The local web server is already running". Although I didn't remember starting the server yet, I tried stopping it using "symfony server:stop" and that crashed my desktop environment.
My theory is that the PID file wasn't deleted because I shut down my system without stopping the web server first and today another process was running using the same ID. So when calling "server:stop" the CLI killed some process related to my desktop environment, thereby crashing it.
Would it make sense to check the process name first, before trying to stop the process? I could work on a pull request if that additional check makes sense.
The text was updated successfully, but these errors were encountered:
this could be a good countermeasure to prevent forcibly terminate legitimate processes.
but this would still leave the processes marked as running in the CLI or have impact on the several checks in place regarding those so I believe we should also try to find a way to make sure this does not happen in the first place.
I'm using Ubuntu. Today, when trying to run "symfony serve" I got a message "The local web server is already running". Although I didn't remember starting the server yet, I tried stopping it using "symfony server:stop" and that crashed my desktop environment.
My theory is that the PID file wasn't deleted because I shut down my system without stopping the web server first and today another process was running using the same ID. So when calling "server:stop" the CLI killed some process related to my desktop environment, thereby crashing it.
I checked the code (https://github.com/symfony-cli/symfony-cli/blob/main/commands/local_server_stop.go) and that seems to be a likely explanation.
Would it make sense to check the process name first, before trying to stop the process? I could work on a pull request if that additional check makes sense.
The text was updated successfully, but these errors were encountered: