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

Fix issue provoked after adding array casting and returning empty fcg… #3724

Merged
merged 1 commit into from
Nov 8, 2023

Conversation

mrodespin
Copy link
Contributor

@mrodespin mrodespin commented Nov 7, 2023

After adding the cast to the array the count($options) > 0 always returns true but with an empty array provoking that the command is executed with an empty parameter which ends in the following issue:

12:33:34 [dev] error in cachetool.php on line 89:
12:33:34 [dev] run cd cachetool.phar opcache:reset --fcgi=
12:33:34 [dev] err In FastCGI.php line 143:
12:33:34 [dev] err FastCGI error: Unable to connect to FastCGI application: Invalid argument (
12:33:34 [dev] err )
12:33:34 [dev] err In Socket.php line 326:
12:33:34 [dev] err Unable to connect to FastCGI application: Invalid argument
12:33:34 [dev] err In Socket.php line 272:
12:33:34 [dev] err stream_socket_client(): Unable to connect to unix:// (Invalid argument)
12:33:34 [dev] err opcache:reset
12:33:34 [dev] exit code 22 (Unknown error)`

@antonmedv antonmedv merged commit cb28eb8 into deployphp:master Nov 8, 2023
midweste pushed a commit to midweste/deployer that referenced this pull request Jan 16, 2024
* upstream/master: (78 commits)
  [automatic] Update docs with bin/docgen
  Move Caddy file in to separate file (deployphp#3765)
  Create parent folders if needed (deployphp#3764)
  [automatic] Update docs with bin/docgen
  Switch to owner change instead of shh user change (deployphp#3763)
  Don't show warning if `become` parameter is set to `root` (deployphp#3761)
  [automatic] Update docs with bin/docgen
  Adding Spiral Framework recipe (deployphp#3582)
  Crontab: allow sudo with crontab:use_sudo option (deployphp#3759)
  fix: run cachetool without args (deployphp#3742)
  Fix version variables in Contao recipe (deployphp#3735)
  contrib: fix creating cron jobs first time
  Removing wrong information regarding reverse DNS (deployphp#3726)
  [automatic] Update docs with bin/docgen
  Fix issue provoked after adding array casting and returning empty fcgi parameter to the command (deployphp#3724)
  Revert "[TASK] Update TYPO3 recipe (deployphp#3674)" (deployphp#3723)
  ci-cd docs: update link to gitlab doc section (deployphp#3721)
  [automatic] Update docs with bin/docgen
  Fix test function (deployphp#3717)
  [automatic] Update docs with bin/docgen
  ...
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
None yet
Projects
None yet
Development

Successfully merging this pull request may close these issues.

2 participants