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

Unset environment variable nginx container health check #823

Merged
merged 1 commit into from Jan 3, 2019
Merged
Changes from all commits
Commits
File filter...
Filter file types
Jump to…
Jump to file or symbol
Failed to load files and symbols.
+4 −4
Diff settings

Always

Just for now

Copy path View file
@@ -1,8 +1,8 @@
#!/bin/sh -v

# cgi-fcgi has issues with very big environment variables.
# LAGOON_ROUTES can get quite long in for lagoon projects with many routes, so we set it to zero (it's not used by cgi-fcgi)
LAGOON_ROUTES=
# cgi-fcgi has issues with non-standard environment variables.
for i in $(env | awk -F"=" '{print $1}') ; do
unset $i ; done

# This script calls the /ping endpoing of the php-fpm, if the return code is 0, the php-fpm has correctly started
SCRIPT_NAME=/${1:-ping} SCRIPT_FILENAME=/${1:-ping} REQUEST_METHOD=GET /usr/bin/cgi-fcgi -bind -connect 127.0.0.1:9000
SCRIPT_NAME=/${1:-ping} SCRIPT_FILENAME=/${1:-ping} REQUEST_METHOD=GET /usr/bin/cgi-fcgi -bind -connect 127.0.0.1:9000
ProTip! Use n and p to navigate between commits in a pull request.