Skip to content
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

Backups fail if shell command not executable #808

Closed
pdewouters opened this issue Apr 27, 2015 · 5 comments
Closed

Backups fail if shell command not executable #808

pdewouters opened this issue Apr 27, 2015 · 5 comments

Comments

@pdewouters
Copy link
Contributor

WordPress 4.2 - backupwordpress Versión 3.2.4. Error: {"php":{"d48b02d27a9fb68b7aa4cd1f88ec329a":"shell_exec(): Unable to execute 'whoami', \/hsphere\/local\/home2\/aarte103\/elsolnews.com\/wp-content\/plugins\/backupwordpress\/functions\/interface.php, 114"}}

See http://stackoverflow.com/questions/11471739/shell-exec-error-when-executing-php-script

@willmot
Copy link
Contributor

willmot commented Apr 27, 2015

That error has nothing to do with the php executable, the error is returned by shell_exec( 'whoami' );, nothing to do with PHP

@pdewouters
Copy link
Contributor Author

Yeah I guess that would only apply if the command runs a php script. so probably the user can't run the whoami command?

@willmot
Copy link
Contributor

willmot commented Apr 27, 2015

yeah, I'm guessing it's that they don't have permission etc. Would go away with #55 anyway so not a huge issue. Although we could at least make sure we are catching the error and not showing the message that uses the output of whoami (which we may already be doing).

@pdewouters pdewouters changed the title Backups fail if PHP executable not in path Backups fail if shell command not executable Apr 29, 2015
@pdewouters
Copy link
Contributor Author

#833 would fix

@willmot
Copy link
Contributor

willmot commented Jul 17, 2015

Going to close as #833 is going in

@willmot willmot closed this as completed Jul 17, 2015
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Projects
None yet
Development

No branches or pull requests

2 participants