fixes #1416

voxpupuli · May 22, 2024 · ba33bcc · ba33bcc
1 parent 7f3a8ae
commit ba33bcc
Show file tree

Hide file tree

Showing 4 changed files with 34 additions and 8 deletions.
diff --git a/manifests/resource/server.pp b/manifests/resource/server.pp
@@ -233,9 +233,10 @@
 #   server stanza, rather than setting a default. Can also be disabled for this
 #   server with the string 'off'.
 # @param error_log
-#   Where to write error log. May add additional options like error level to
-#   the end. May set to 'absent', in which case it will be omitted in this
+#   Where to write error log. May set to 'absent', in which case it will be omitted in this
 #   server stanza (and default to nginx.conf setting)
+# @param error_log_severity
+#   Optional error level. Defaults to nginx.conf setting
 # @param passenger_cgi_param
 #   Allows one to define additional CGI environment variables to pass to the backend application
 # @param passenger_set_header
@@ -399,6 +400,7 @@
   Optional[Array[String]] $include_files                                         = undef,
   Optional[Variant[String, Array]] $access_log                                   = undef,
   Optional[Variant[String, Array]] $error_log                                    = undef,
+  Optional[Nginx::ErrorLogSeverity] $error_log_severity                          = $nginx::nginx_error_log_severity,
   Optional[String] $format_log                                                   = $nginx::http_format_log,
   Optional[Hash] $passenger_cgi_param                                            = undef,
   Optional[Hash] $passenger_set_header                                           = undef,

diff --git a/spec/defines/resource_server_spec.rb b/spec/defines/resource_server_spec.rb
@@ -358,6 +358,18 @@
                 '  error_log             syslog:server=localhost;'
               ]
             },
+            {
+              title: 'should set error_log severity level',
+              attr: 'error_log_severity',
+              value: 'warn',
+              match: '  error_log             /path/to/error.log warn;'
+            },
+            {
+              title: 'should not set error_log severity level',
+              attr: 'error_log_severity',
+              value: :undef,
+              match: '  error_log             /path/to/error.log;'
+            },
             {
               title: 'should not include error_log in server when set to absent',
               attr: 'error_log',
@@ -1017,6 +1029,18 @@
                 '  error_log             syslog:server=localhost;'
               ]
             },
+            {
+              title: 'should set error_log severity level',
+              attr: 'error_log_severity',
+              value: 'warn',
+              match: '  error_log             /path/to/error.log warn;'
+            },
+            {
+              title: 'should not set error_log severity level',
+              attr: 'error_log_severity',
+              value: :undef,
+              match: '  error_log             /path/to/error.log;'
+            },
             {
               title: 'should not include error_log in server when set to absent',
               attr: 'error_log',

diff --git a/templates/server/server_header.erb b/templates/server/server_header.erb
@@ -200,13 +200,13 @@ server {
 <% end -%>
 <% if @error_log.is_a?(Array) -%>
   <%- @error_log.each do |log_item| -%>
-  error_log             <%= log_item %>;
+  error_log             <%= log_item %><% if @error_log_severity %> <%= @error_log_severity %><% end %>;
   <%- end -%>
 <% elsif @error_log == 'absent' -%>
 <% elsif not @error_log -%>
-  error_log             <%= scope['::nginx::config::log_dir'] %>/<%= @name_sanitized %>.error.log;
+  error_log             <%= scope['::nginx::config::log_dir'] %>/<%= @name_sanitized %>.error.log<% if @error_log_severity %> <%= @error_log_severity %><% end %>;
 <% else -%>
-  error_log             <%= @error_log %>;
+  error_log             <%= @error_log %><% if @error_log_severity %> <%= @error_log_severity %><% end %>;
 <% end -%>
 <% if @error_pages -%>
 

diff --git a/templates/server/server_ssl_header.erb b/templates/server/server_ssl_header.erb
@@ -129,13 +129,13 @@ server {
 <% end -%>
 <% if @error_log.is_a?(Array) -%>
   <%- @error_log.each do |log_item| -%>
-  error_log             <%= log_item %>;
+  error_log             <%= log_item %><% if @error_log_severity %> <%= @error_log_severity %><% end %>;
   <%- end -%>
 <% elsif @error_log == 'absent' -%>
 <% elsif not @error_log -%>
-  error_log             <%= scope['::nginx::config::log_dir'] %>/ssl-<%= @name_sanitized %>.error.log;
+  error_log             <%= scope['::nginx::config::log_dir'] %>/ssl-<%= @name_sanitized %>.error.log<% if @error_log_severity %> <%= @error_log_severity %><% end %>;
 <% else -%>
-  error_log             <%= @error_log %>;
+  error_log             <%= @error_log %><% if @error_log_severity %> <%= @error_log_severity %><% end %>;
 <% end -%>
 <% if @error_pages -%>