Make start-nginx scripts ignore the SIGTERM signal so they do not kil…

…l off nginx abruptly.
heroku · Aug 6, 2021 · 40ff4d2 · 40ff4d2
1 parent 14ccd81
commit 40ff4d2
Show file tree

Hide file tree

Showing 4 changed files with 7 additions and 0 deletions.
diff --git a/bin/start-nginx b/bin/start-nginx
@@ -1,5 +1,7 @@
 #!/usr/bin/env bash
 
+trap "echo start-nginx got SIGTERM. Ignoring while nginx shuts down gracefully" SIGTERM
+
 psmgr=/tmp/nginx-buildpack-wait
 rm -f $psmgr
 mkfifo $psmgr

diff --git a/bin/start-nginx-debug b/bin/start-nginx-debug
@@ -1,5 +1,7 @@
 #!/usr/bin/env bash
 
+trap "echo start-nginx-debug got SIGTERM. Ignoring while nginx shuts down gracefully" SIGTERM
+
 psmgr=/tmp/nginx-buildpack-wait
 rm -f $psmgr
 mkfifo $psmgr

diff --git a/bin/start-nginx-solo b/bin/start-nginx-solo
@@ -1,5 +1,7 @@
 #!/usr/bin/env bash
 
+trap "echo start-nginx-solo got SIGTERM. Ignoring while nginx shuts down gracefully" SIGTERM
+
 psmgr=/tmp/nginx-buildpack-wait
 rm -f $psmgr
 mkfifo $psmgr

diff --git a/changelog.md b/changelog.md
@@ -7,6 +7,7 @@ and this project adheres to [Semantic Versioning](https://semver.org/spec/v2.0.0
 ## [x.x] - xxxx-xx-xx
 ### Changes
 - [all] Patched nginx so it gracefully shuts down on SIGTERM.
+- [all] start-nginx scripts ignore the SIGTERM signal so they do not kill off nginx abruptly.
 
 ## [1.7] - 2021-06-04
 ### Changes