Undefined error if nginx is not installed

I have setup a new droplet without nginx installed.

If i run ghost install, i am getting this error:

✖ Running setup checks
A ProcessError occured.

Error occurred running command: 'undefined'

I debugged quickly, here is the output:

System Stack{ ListrError: Something went wrong
    at ListrError (/root/ghost-cli/node_modules/listr/lib/listr-error.js:4:3)
    at tasks.then (/root/ghost-cli/node_modules/listr/index.js:103:18)
    at process._tickCallback (internal/process/next_tick.js:109:7)
  name: 'ListrError',
  errors:
   [ { Error: Command failed: /bin/sh -c dpkg -l | grep nginx


         at Promise.all.then.arr (/root/ghost-cli/node_modules/execa/index.js:231:11)
         at process._tickCallback (internal/process/next_tick.js:109:7)
       code: 1,
       killed: false,
       stdout: '',
       stderr: '',
       failed: true,
       signal: null,
       cmd: '/bin/sh -c dpkg -l | grep nginx',
       timedOut: false } ],
  context:
   { setup: true,
     ui:
      UI {
        options: [Object],
        stdin: [Object],
        stdout: [Object],
        stderr: [Object],
        verbose: false,
        allowPrompt: true,
        inquirer: [Object],
        spinner: [Object] },
     listr:
      Listr {
        _options: [Object],
        _tasks: [Object],
        concurrency: Infinity,
        _RendererClass: [Function: SilentRenderer],
        exitOnError: false,
        _renderer: SilentRenderer {} },
     linux: true,
     ubuntu: true,
     systemd: true } }

I expect that the CLI tells me that nginx is not installed and skips this extension. The problem here is that dpkg -l | grep nginx returns exit code 1, which results in an error.

We need two things here:

• somehow handle exit code 1, which is the default exit code of dpkg if a package could not be found (either as a hack in the CLI or push a PR to execa)
• ensure the error messages shows: “Nginx not installed/found.”

Steps to reproduce:

1. create a droplet without nginx
2. run ghost install

successful debugging

I have added a super hack:

            if (!(error instanceof errors.SystemError)) {
                // if nginx is not installed, exit code is 1
                if (error.errors && error.errors.length && error.errors[0].code !== 1) {
                    return Promise.reject(new errors.ProcessError(error));
                } else {
                    error.message = 'Nginx is not installed';
                }
            }

into https://github.com/TryGhost/Ghost-CLI/blob/master/lib/commands/doctor/checks/setup.js#L44, which worked for me.

CLI: latest master

[acburdine]

oh yeah looking at those checks the error handling is all kinds of crap - will fix it 😄

[acburdine]

ah crap, forgot about that 😕