From: Alexander Moisseev <moiseev@mezonplus.ru>
Date: Fri, 5 Jun 2026 09:18:26 +0000 (+0300)
Subject: [Minor] url_redirector: clarify log messages for successful HTTP responses
X-Git-Tag: 4.1.0~1^2
X-Git-Url: http://git.ipfire.org/gitweb/index.cgi?a=commitdiff_plain;h=e4c177b4d4c6e8e795c44c8ae4a1d930652a8b02;p=thirdparty%2Frspamd.git

[Minor] url_redirector: clarify log messages for successful HTTP responses

The phrases "err code 200" and "err code <N>" 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.
---

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