-
Notifications
You must be signed in to change notification settings - Fork 310
New issue
Have a question about this project? Sign up for a free GitHub account to open an issue and contact its maintainers and the community.
By clicking “Sign up for GitHub”, you agree to our terms of service and privacy statement. We’ll occasionally send you account related emails.
Already on GitHub? Sign in to your account
Have psysh break into a web interface #505
Comments
Yep, this is a great idea! Wanna open a pull request? 😉 |
Well, I have a lot to learn about the PsySH codebase before I can usefully contribute. 😉 In particular, I'm not sure how the forking loop is implemented, does it fork when PHP runs as a web server, or does it always fork? |
It always forks, if your PHP install has the pcntl and posix extensions. If you don't have those two extensions (i.e. Windows but not WSL or Cygwin users) it has a non-forking mode which doesn't fork at all… which is a bit less awesome, because uncatchable fatal errors drop you back out to the command line rather than continuing. But we can probably ignore these users, as I doubt we'd be able to implement in-browser breakpoints without forks. For everyone else, basically:
|
Maybe you could also collaborate with Whoops which is error handler framework for PHP. It also include user friendly error handler page, but it currently doesn't have REPL. Maybe you could collaborate with them and integrate PsySH into their error handlers (or opposite). See filp/whoops#585. |
I would love having something like this in whoops, but I guess it needs to be some kind of external optional feature because it might double the codebase in terms of lines of code I guess? |
@staabm Yes, this could be a problem. There are two possible solutions. First is to have it as external optional feature like you said and second is to maybe join PsySH and Whoops so they will have some shared code. |
https://github.com/spatie/laravel-web-tinker is related to this. |
This would very significantly change the security footprint of PsySH. Right now there isn't much risk in making it available on a production server - anyone who can get a shell on the server and can execute PHP can already do anything PsySH can do, just less comfortably. But getting access to a web port is much easier and much harder to control than getting shell access; I would be very uncomfortable with code for a remote code execution backdoor being present on production servers, even if the feature is off by default - from there on it's just one small mistake, either by the PsySH developers or by whoever is configuring PsySH for the given application, and anyone who can get a HTTP connection can run code on the server. IMO if a feature like this gets implemented, it should be in a separate Composer package so the code only gets installed on servers where it's wanted. |
Recently, when working with the Python Flask framework, I discovered a cool feature — it allows you to run code inside the web browser:
Could we have the PsySH breakpoint launch a web interface? Not having to switch back-and-forth between the terminal and the browser would be a great plus.
The text was updated successfully, but these errors were encountered: