Update logging (#87)

* add more test logging * update error_log directive in Docker Nginx config to match real BOSH job configuration * update log for content-type behavior to use ngx.NOTICE level so that it gets logged correctly
cloud-gov · Jun 26, 2024 · 5ca31bc · 5ca31bc
1 parent 50f3187
commit 5ca31bc
Show file tree

Hide file tree

Showing 2 changed files with 4 additions and 4 deletions.
diff --git a/docker/nginx.conf b/docker/nginx.conf
@@ -1,6 +1,6 @@
 worker_processes 4;
 
-error_log stderr debug;
+error_log stderr notice;
 
 worker_rlimit_nofile 40000;
 
@@ -119,7 +119,7 @@ http {
 
       header_filter_by_lua_block {
         if ngx.var.upstream_http_content_type == nil and ngx.var.mapped_content_type == "" then
-          ngx.log(ngx.INFO, "no Content-Type header")
+          ngx.log(ngx.NOTICE, "no Content-Type header was mapped for upstream status code ", ngx.var.upstream_status)
         end
       }
 

diff --git a/jobs/secureproxy/templates/config/nginx.conf.erb b/jobs/secureproxy/templates/config/nginx.conf.erb
@@ -197,7 +197,7 @@ http {
 
       header_filter_by_lua_block {
         if ngx.var.upstream_http_content_type == nil and ngx.var.mapped_content_type == "" then
-          ngx.log(ngx.INFO, "no Content-Type header")
+          ngx.log(ngx.NOTICE, "no Content-Type header was mapped for upstream status code ", ngx.var.upstream_status)
         end
       }
 
@@ -294,7 +294,7 @@ server {
 
     header_filter_by_lua_block {
       if ngx.var.upstream_http_content_type == nil and ngx.var.mapped_content_type == "" then
-        ngx.log(ngx.INFO, "no Content-Type header")
+        ngx.log(ngx.NOTICE, "no Content-Type header was mapped for upstream status code ", ngx.var.upstream_status)
       end
     }