Validate whether php-fpm has status page enabled

PHP fpm will return "File not found" if the `pm.status_path` is
configured to `/status`, we check that in the script and provide an exit
status of 8.

Author: renatomefi

php-fpm-healthcheck

@@ -18,7 +18,8 @@
 # Ping mode with data (outputs php-fpm status text): ./php-fpm-healthcheck -v
 #
 # Exit status codes:
-# 9 - Couldn't reach PHP fpm status page, have you configured it with `pm.status_path = /status`?
+# 9 - Couldn't connect to PHP fpm, is it running?
+# 8 - Couldn't reach PHP fpm status page, have you configured it with `pm.status_path = /status`?
 # 1 - A healthcheck condition has failed
 # 3 - Invalid option given
 # 4 - One or more required softwares are missing
@@ -55,10 +56,15 @@ get_fpm_status() {
     if test "$VERBOSE" = 1; then printf "Trying to connect to php-fpm via: %s\\n" "$1"; fi;
 
     # Since I cannot use pipefail I'll just split these in two commands
-    FPM_STATUS=$(cgi-fcgi -bind -connect "$1")
+    FPM_STATUS=$(cgi-fcgi -bind -connect "$1" 2> /dev/null)
     FPM_STATUS=$(echo "$FPM_STATUS" | tail +5)
 
     if test "$VERBOSE" = 1; then printf "php-fpm status output:\\n%s\\n" "$FPM_STATUS"; fi;
+
+    if test "$FPM_STATUS" = "File not found."; then
+        >&2 printf "php-fpm status page non reachable\\n";
+        exit 8;
+    fi;
 }
 
 # $1 - fpm option

test/testinfra/test_fpm.py

@@ -0,0 +1,41 @@
+import pytest
+
+@pytest.mark.php_fpm
+def test_exit_when_no_status_page_is_configured(host):
+    # disable fpm status page
+    host.run("sed -i /usr/local/etc/php-fpm.d/zz-docker.conf -e '/pm.status_path/ s/^;*/;/'")
+    host.run("kill -USR2 1")
+    
+    cmd = host.run("php-fpm-healthcheck -v")
+    assert cmd.rc == 8
+    assert "Trying to connect to php-fpm via:" in cmd.stdout
+    assert "status output:" in cmd.stdout
+    assert "php-fpm status page non reachable" in cmd.stderr
+
+    # enable fpm status page back
+    host.run("sed -i /usr/local/etc/php-fpm.d/zz-docker.conf -e '/;pm.status_path/ s/^;*//'")
+    host.run("kill -USR2 1")
+
+@pytest.mark.alpine
+def test_exit_when_fpm_is_not_reachable_apk(host):
+    cmd = host.run("FCGI_CONNECT=localhost:9001 php-fpm-healthcheck -v")
+    assert cmd.rc == 9
+    assert "Trying to connect to php-fpm via: localhost:9001" in cmd.stdout
+
+@pytest.mark.alpine
+def test_exit_when_fpm_is_invalid_host_apk(host):
+    cmd = host.run("FCGI_CONNECT=abc php-fpm-healthcheck -v")
+    assert cmd.rc == 9
+    assert "Trying to connect to php-fpm via: abc" in cmd.stdout
+
+@pytest.mark.stretch
+def test_exit_when_fpm_is_not_reachable_apt(host):
+    cmd = host.run("FCGI_CONNECT=localhost:9001 php-fpm-healthcheck -v")
+    assert cmd.rc == 111
+    assert "Trying to connect to php-fpm via: localhost:9001" in cmd.stdout
+
+@pytest.mark.stretch
+def test_exit_when_fpm_is_invalid_host_apt(host):
+    cmd = host.run("FCGI_CONNECT=abc php-fpm-healthcheck -v")
+    assert cmd.rc == 2
+    assert "Trying to connect to php-fpm via: abc" in cmd.stdout