roles: update opts on config reload

There were no way to update config on the run with method
`config:reload()` instead of listen parameters.

This patch fixes it, all options support config reload.

Closes #216

Author: themilchenko

CHANGELOG.md

@@ -13,6 +13,7 @@ and this project adheres to [Semantic Versioning](http://semver.org/spec/v2.0.0.
 ## Fixed
 
 - Do not recreate server if it's address and port were not changed (#219).
+- Server doesn't change after updating parameters on config reload (#216).
 
 ## [1.8.0] - 2025-07-07
 

roles/httpd.lua

@@ -107,23 +107,37 @@ local function parse_params(node)
     }
 end
 
+local function server_has_changed(name, node_params, host, port)
+    if servers[name].httpd.host ~= host or servers[name].httpd.port ~= port then
+        return true
+    end
+    for k in pairs(node_params) do
+        if k ~= 'listen' and servers[name].httpd.options[k] ~= node_params[k] then
+            return true
+        end
+    end
+    return false
+end
+
 local function apply_http(name, node)
     local host, port, err = parse_listen(node.listen)
     if err ~= nil then
         error("failed to parse URI: " .. err)
     end
 
+    local params = parse_params(node)
+
     if servers[name] == nil then
-        local httpd = http_server.new(host, port, parse_params(node))
+        local httpd = http_server.new(host, port, params)
 
         httpd:start()
         servers[name] = {
             httpd = httpd,
             routes = {},
         }
-    elseif servers[name].httpd.host ~= host or servers[name].httpd.port ~= port then
+    elseif server_has_changed(name, params, host, port) then
         servers[name].httpd:stop()
-        servers[name].httpd = http_server.new(host, port, parse_params(node))
+        servers[name].httpd = http_server.new(host, port, params)
         servers[name].httpd:start()
     end
 end

test/integration/httpd_role_test.lua

@@ -233,3 +233,25 @@ for log_name, log_lvl in pairs(LOG_LEVELS) do
         assert_should_log(log_level >= LOG_LEVELS.INFO)
     end
 end
+
+g.test_enable_tls_on_config_reload = function(cg)
+    -- We should start with no tls firstly.
+    t.skip_if(cg.params.use_tls)
+
+    local resp = http_client:get('http://localhost:13000/ping')
+    t.assert_equals(resp.status, 200, 'response not 200')
+    t.assert_equals(resp.body, 'pong')
+
+    treegen.write_file(cg.server.chdir, 'config.yaml', yaml.encode(tls_config))
+    local _, err = cg.server:eval("require('config'):reload()")
+    t.assert_not(err)
+
+    resp = http_client:get('https://localhost:13000/ping', {
+        ca_file = fio.pathjoin(ssl_data_dir, 'ca.crt')
+    })
+    t.assert_equals(resp.status, 200, 'response not 200')
+    t.assert_equals(resp.body, 'pong')
+
+    local resp = http_client:get('http://localhost:13000/ping')
+    t.assert_equals(resp.status, 444, 'response not 444')
+end