RepoMirrors
/
haproxy
mirror of http://git.haproxy.org/git/haproxy.git/
1
0
Fork 0
Code Issues Packages Projects Releases Wiki Activity

BUG/MINOR: cli: don't show sockpairs in HAPROXY_CLI and HAPROXY_MASTER_CLI 

Before this fix, HAPROXY_CLI and HAPROXY_MASTER_CLI have contained along with
CLI sockets addresses internal sockpairs, which are used only for master CLI
(reload sockpair and sockpair shared with a worker process). These internal
sockpairs are always need to be hidden.

At the moment there is no any client, who uses sockpair addresses for the
stats listener or in order to connect to master CLI. So, let's simply not copy
these internal sockpair addresses of MASTER and GLOBAL proxy listeners.

As listeners with sockpairs are skipped and they can be presented in the
listeners list in any order, let's add semicolon separator between addresses
only in the case, when there are already some string saved in the trash and we
are sure, that we are adding a new address to it. Otherwise, we could have such
weird output:

	HAPROXY_MASTER_CLI=unix@/tmp/mcli.sock;;

This fix is need to be backported in all stable versions.
This commit is contained in:
Valentine Krasnobaeva 2024-11-12 22:43:49 +01:00 committed by William Lallemand
parent 1f0cd91fe7
commit 113745e6f0
1 changed files with 7 additions and 10 deletions
Show Stats Download Patch File Download Diff File Expand all files Collapse all files

17
src/cli.c
View File

@ -651,10 +651,6 @@ int listeners_setenv(struct proxy *frontend, const char *varname)
char addr[46];
char port[6];
/* separate listener by semicolons */
if (trash->data)
chunk_appendf(trash, ";");
if (l->rx.addr.ss_family == AF_UNIX ||
l->rx.addr.ss_family == AF_CUST_ABNS ||
l->rx.addr.ss_family == AF_CUST_ABNSZ) {
@ -663,21 +659,22 @@ int listeners_setenv(struct proxy *frontend, const char *varname)
un = (struct sockaddr_un *)&l->rx.addr;
if (l->rx.addr.ss_family == AF_CUST_ABNS ||
l->rx.addr.ss_family == AF_CUST_ABNSZ) {
chunk_appendf(trash, "abns@%s", un->sun_path+1);
chunk_appendf(trash, "%sabns@%s", (trash->data ? ";" : ""), un->sun_path+1);
} else {
chunk_appendf(trash, "unix@%s", un->sun_path);
chunk_appendf(trash, "%sunix@%s", (trash->data ? ";" : ""), un->sun_path);
}
} else if (l->rx.addr.ss_family == AF_INET) {
addr_to_str(&l->rx.addr, addr, sizeof(addr));
port_to_str(&l->rx.addr, port, sizeof(port));
chunk_appendf(trash, "ipv4@%s:%s", addr, port);
chunk_appendf(trash, "%sipv4@%s:%s", (trash->data ? ";" : ""), addr, port);
} else if (l->rx.addr.ss_family == AF_INET6) {
addr_to_str(&l->rx.addr, addr, sizeof(addr));
port_to_str(&l->rx.addr, port, sizeof(port));
chunk_appendf(trash, "ipv6@[%s]:%s", addr, port);
} else if (l->rx.addr.ss_family == AF_CUST_SOCKPAIR) {
chunk_appendf(trash, "sockpair@%d", ((struct sockaddr_in *)&l->rx.addr)->sin_addr.s_addr);
chunk_appendf(trash, "%sipv6@[%s]:%s", (trash->data ? ";" : ""), addr, port);
}
/* AF_CUST_SOCKPAIR is explicitly skipped, we don't want to show reload and shared
* master CLI sockpairs in HAPROXY_CLI and HAPROXY_MASTER_CLI
*/
}
}
trash->area[trash->data++] = '\0';