Merge pull request #137 from noris-network/max_request_time

parametrize max_request_time

README.md

@@ -94,6 +94,9 @@ Use correct backslash escaping in unlang. Default: `true`
 ##### `max_requests`
 The maximum number of requests which the server keeps track of. This should be 256 multiplied by the number of clients. Default: `4096`
 
+##### `max_request_time`
+The maximum time (in seconds) to handle a request. Default: `30`
+
 ##### `max_servers`
 Limit on the total number of servers running. Default: `4096`
 

manifests/init.pp

@@ -3,6 +3,7 @@ class freeradius (
   Boolean $control_socket          = false,
   Integer $max_servers             = 4096,
   Integer $max_requests            = 4096,
+  Integer $max_request_time        = 30,
   Boolean $mysql_support           = false,
   Boolean $pgsql_support           = false,
   Boolean $perl_support            = false,

spec/classes/freeradius_spec.rb

@@ -24,6 +24,7 @@ describe 'freeradius' do
       #:control_socket => false,
       #:max_servers => "4096",
       #:max_requests => "4096",
+      #:max_request_time => "30",
       #:mysql_support => false,
       #:perl_support => false,
       #:utils_support => false,

spec/shared_contexts.rb

@@ -13,6 +13,7 @@ shared_context :global_hiera_data do
        #"freeradius::ldap_support" => '',
        #"freeradius::krb5_support" => '',
        #"freeradius::max_requests" => '',
+       #"freeradius::max_request_time" => '',
        #"freeradius::max_servers" => '',
        #"freeradius::mysql_support" => '',
        #"freeradius::perl_support" => '',

templates/radiusd.conf.erb

@@ -190,7 +190,7 @@ correct_escapes = <%= @correct_escapes %>
 #
 #  Useful range of values: 5 to 120
 #
-max_request_time = 30
+max_request_time = <%= @max_request_time %>
 
 #  cleanup_delay: The time to wait (in seconds) before cleaning up
 #  a reply which was sent to the NAS.