Documentation - add notice to check the location of binaries

antonioribeiro / health

Laravel Health Panel

BSD 3-Clause "New" or "Revised" License

1.95k stars 197 forks source link

Documentation - add notice to check the location of binaries #108

Closed JustinGT closed 5 years ago

JustinGT commented 5 years ago

In the .yml files you have

name: Latency
.
.
.
binary: /sbin/ping
.
.
.

Depending upon you os flavour, it may not be in that folder:

health.test$ which ping
/bin/ping

So if you could make a note telling people to check that, it would probably save some of the "Its not working" issues!

antonioribeiro commented 5 years ago

Okay, thanks for reporting. Added a way to configure binaries via .env, since our local dev and production may differ:

'services' => [
    'ping' => [
        'bin' => env('HEALTH_PING_BIN', '/sbin/ping'),
    ],

    'composer' => [
        'bin' => env('HEALTH_COMPOSER_BIN', 'composer'),
    ],
],