[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 262ad59f8b8c528b63f4c866e70c31116cb19d1b..257c60be0be1dc7bae2198f89726782be644508f 100644 (file)
--- 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