]> git.ipfire.org Git - thirdparty/haproxy.git/commitdiff
MINOR: protocol: uniformize protocol errors
authorWilly Tarreau <w@1wt.eu>
Thu, 14 Oct 2021 09:59:15 +0000 (11:59 +0200)
committerWilly Tarreau <w@1wt.eu>
Thu, 14 Oct 2021 19:22:52 +0000 (21:22 +0200)
Some protocols fail with "error blah [ip:port]" and other fail with
"[ip:port] error blah". All this already appears in a "starting" or
"binding" context after a proxy name. Let's choose a more universal
approach like below where the ip:port remains at the end of the line
prefixed with "for".

  [WARNING]  (18632) : Binding [binderr.cfg:10] for proxy http: cannot bind receiver to device 'eth2' (No such device) for [0.0.0.0:1080]
  [WARNING]  (18632) : Starting [binderr.cfg:10] for proxy http: cannot set MSS to 12 for [0.0.0.0:1080]

src/proto_quic.c
src/proto_sockpair.c
src/proto_tcp.c
src/proto_udp.c
src/proto_uxdg.c
src/proto_uxst.c
src/sock_inet.c

index 257a49204b74a52f0cc49fbbdf77bf214ba5dcfb..58679eb3abdb06986a4d0496d0971f5ade8e6c9f 100644 (file)
@@ -607,7 +607,7 @@ static int quic_bind_listener(struct listener *listener, char *errmsg, int errle
                char pn[INET6_ADDRSTRLEN];
 
                addr_to_str(&listener->rx.addr, pn, sizeof(pn));
-               snprintf(errmsg, errlen, "%s [%s:%d]", msg, pn, get_host_port(&listener->rx.addr));
+               snprintf(errmsg, errlen, "%s for [%s:%d]", msg, pn, get_host_port(&listener->rx.addr));
        }
        return err;
 }
index 216700b40dde8910a97a1537fb94f3cd666aa094..11cf3537faf2537ed927e72f4c641d06917763d7 100644 (file)
@@ -162,7 +162,7 @@ int sockpair_bind_receiver(struct receiver *rx, char **errmsg)
 
  bind_return:
        if (errmsg && *errmsg)
-               memprintf(errmsg, "%s [fd %d]", *errmsg, rx->fd);
+               memprintf(errmsg, "%s for [fd %d]", *errmsg, rx->fd);
 
        return err;
 
index 95466538b23c46498c7ec9a3070858d68bbd3414..18e9b80e492d04cf6057cd504055709044046d2e 100644 (file)
@@ -734,7 +734,7 @@ int tcp_bind_listener(struct listener *listener, char *errmsg, int errlen)
                char pn[INET6_ADDRSTRLEN];
 
                addr_to_str(&listener->rx.addr, pn, sizeof(pn));
-               snprintf(errmsg, errlen, "[%s:%d]: %s", pn, get_host_port(&listener->rx.addr), msg->area);
+               snprintf(errmsg, errlen, "%s for [%s:%d]", msg->area, pn, get_host_port(&listener->rx.addr));
        }
        free_trash_chunk(msg);
        msg = NULL;
index f07170f297f1328e8d397129a5fe020e14b3d9e4..c5ecfacd0fd5277de145313750ad71739af675f0 100644 (file)
@@ -148,7 +148,7 @@ int udp_bind_listener(struct listener *listener, char *errmsg, int errlen)
                char pn[INET6_ADDRSTRLEN];
 
                addr_to_str(&listener->rx.addr, pn, sizeof(pn));
-               snprintf(errmsg, errlen, "%s [%s:%d]", msg, pn, get_host_port(&listener->rx.addr));
+               snprintf(errmsg, errlen, "%s for [%s:%d]", msg, pn, get_host_port(&listener->rx.addr));
        }
        return err;
 }
index 7eb5a6252af116086cae906ad7965b1fa9afb239..b0efe867c494afc6349dcf234a969bb3f3602e5d 100644 (file)
@@ -103,7 +103,7 @@ int uxdg_bind_listener(struct listener *listener, char *errmsg, int errlen)
  uxdg_return:
        if (msg && errlen) {
                const char *path = ((struct sockaddr_un *)&listener->rx.addr)->sun_path;
-                snprintf(errmsg, errlen, "%s [%s]", msg, path);
+                snprintf(errmsg, errlen, "%s for [%s]", msg, path);
        }
        return err;
 }
index 47a9992435a115b72a2464166cf2d1f24c821861..45cb56f369ae193313bf7fe201e7686379df59f1 100644 (file)
@@ -141,7 +141,7 @@ static int uxst_bind_listener(struct listener *listener, char *errmsg, int errle
  uxst_return:
        if (msg && errlen) {
                const char *path = ((struct sockaddr_un *)&listener->rx.addr)->sun_path;
-               snprintf(errmsg, errlen, "%s [%s]", msg, path);
+               snprintf(errmsg, errlen, "%s for [%s]", msg, path);
        }
        return err;
 }
index 1402dd076d79831555014996e58f1ca36e9b867c..fd8f64893333409148e31a5463f0bb6ca8689c7c 100644 (file)
@@ -402,7 +402,7 @@ int sock_inet_bind_receiver(struct receiver *rx, char **errmsg)
                char pn[INET6_ADDRSTRLEN];
 
                addr_to_str(&addr_inet, pn, sizeof(pn));
-               memprintf(errmsg, "%s [%s:%d]", *errmsg, pn, get_host_port(&addr_inet));
+               memprintf(errmsg, "%s for [%s:%d]", *errmsg, pn, get_host_port(&addr_inet));
        }
        return err;