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

PHPCS throws errors determining auto report width when shell_exec is disabled #2698

Closed
linux4me opened this issue Nov 14, 2019 · 0 comments · Fixed by #2733
Closed

PHPCS throws errors determining auto report width when shell_exec is disabled #2698

linux4me opened this issue Nov 14, 2019 · 0 comments · Fixed by #2733
Milestone

Comments

@linux4me
Copy link

If a server has shell_exec() disabled for security reasons, the error logs fill up with warnings like this each time a PHP CodeSniffer command is run:
PHP Warning: shell_exec() has been disabled for security reasons in /.composer/vendor/squizlabs/php_codesniffer/src/Config.php on line 222
Disabling shell_exec() doesn't interfere with PHP CodeSniffer's functioning, but a simple check to see if shell_exec() is disabled with function_exists() would prevent the warning on servers that disable shell_exec().

@gsherwood gsherwood added this to the 3.5.4 milestone Dec 3, 2019
@gsherwood gsherwood changed the title Shell_exec() Used In /src/Config.php on line 222 PHPCS throws errors when shell_exec is disabled Dec 8, 2019
@gsherwood gsherwood changed the title PHPCS throws errors when shell_exec is disabled PHPCS throws errors determining auto report width when shell_exec is disabled Dec 8, 2019
gsherwood added a commit that referenced this issue Dec 8, 2019
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Projects
None yet
Development

Successfully merging a pull request may close this issue.

2 participants