From e4c177b4d4c6e8e795c44c8ae4a1d930652a8b02 Mon Sep 17 00:00:00 2001 From: Alexander Moisseev Date: Fri, 5 Jun 2026 12:18:26 +0300 Subject: [PATCH] [Minor] url_redirector: clarify log messages for successful HTTP responses The phrases "err code 200" and "err code " are misleading since they refer to HTTP status codes, not errors. Successful resolutions (HTTP 200) and intermediate redirects (30x) now use unambiguous wording that clearly separates the action from the status code. --- src/plugins/lua/url_redirector.lua | 6 +++--- 1 file changed, 3 insertions(+), 3 deletions(-) diff --git a/src/plugins/lua/url_redirector.lua b/src/plugins/lua/url_redirector.lua index 262ad59f8b..257c60be0b 100644 --- a/src/plugins/lua/url_redirector.lua +++ b/src/plugins/lua/url_redirector.lua @@ -593,10 +593,10 @@ http_walk = function(task, orig_url, url, ntries, chain, seen) if code == 200 then if orig_url == url then - rspamd_logger.infox(task, 'direct url %s, err code 200', url) + rspamd_logger.infox(task, 'url %s resolved directly (HTTP 200)', url) else rspamd_logger.infox(task, - 'found redirect from %s to %s, err code 200', orig_url, url) + 'redirect chain resolved: %s -> %s (HTTP 200)', orig_url, url) end chain_append(chain, url) finalize_chain(task, chain, nil) @@ -616,7 +616,7 @@ http_walk = function(task, orig_url, url, ntries, chain, seen) 'failed to parse redirect location even after encoding: %s', loc) end end - lua_util.debugm(N, task, 'found redirect from %s to %s, err code %s', + lua_util.debugm(N, task, 'redirect from %s to %s (HTTP %s)', orig_url, loc, code) -- 'url' just returned 30x, so it's an intermediate. Save it -- 2.47.3