From: Willy Tarreau Date: Mon, 16 Jan 2023 12:55:27 +0000 (+0100) Subject: MINOR: listener: also support "quic+" as an address prefix X-Git-Tag: v2.8-dev2~32 X-Git-Url: http://git.ipfire.org/cgi-bin/gitweb.cgi?a=commitdiff_plain;h=40725a4eb0beadcdaa3c60b6f8c8fe20ab53abc3;p=thirdparty%2Fhaproxy.git MINOR: listener: also support "quic+" as an address prefix While we do support quic4@ and quic6@ for listening addresses, it was not possible to specify that we want to use an FD inherited from the parent with QUIC. It's just a matter of making it possible to enable a dgram-type socket and a stream-type transport, so let's add this. Now it becomes possible to write "quic+fd@12", "quic+ipv4@addr" etc. --- diff --git a/doc/configuration.txt b/doc/configuration.txt index 3054029fbd..7ee08acc32 100644 --- a/doc/configuration.txt +++ b/doc/configuration.txt @@ -24078,8 +24078,11 @@ this address but in some cases the user may force it to a different one. This is the case for "log" statement where the default is syslog over UDP but we could force to use syslog over TCP. -Those prefixes were designed for internal purpose and users should -instead use aliases of the next section "11.3 Protocol prefixes". +Those prefixes were designed for internal purpose and users should instead use +use aliases of the next section "11.3 Protocol prefixes". However these can +sometimes be convenient, for example in combination with inherited sockets +known by their file descriptor number, in which case the address family is "fd" +and the socket type must be declared. If users need one those prefixes to perform what they expect because they can not configure the same using the protocol prefixes, they should @@ -24091,6 +24094,10 @@ report this to the maintainers. 'dgram+@
' forces socket type and transport method to "datagram". +'quic+@
' forces socket type to "datagram" and transport + method to "stream". + + 11.3 Protocol prefixes ---------------------- @@ -24100,14 +24107,16 @@ report this to the maintainers. "datagram" and the transport method is forced to "stream". Depending on the statement using this address, a UDP port or port range can or - must be specified. + must be specified. It is equivalent to + "quic+ipv4@". 'quic6@
[:port1[-port2]]' following
is always considered as an IPv6 address but socket type is forced to "datagram" and the transport method is forced to "stream". Depending on the statement using this address, a UDP port or port range can or - must be specified. + must be specified. It is equivalent to + "quic+ipv6@". 'tcp@
[:port1[-port2]]' following
is considered as an IPv4 or IPv6 address depending of the syntax but diff --git a/src/tools.c b/src/tools.c index 1f4fedaaaa..f7fdfff814 100644 --- a/src/tools.c +++ b/src/tools.c @@ -1002,6 +1002,11 @@ struct sockaddr_storage *str2sa_range(const char *str, int *port, int *low, int proto_type = PROTO_TYPE_DGRAM; ctrl_type = SOCK_DGRAM; } + else if (strncmp(str2, "quic+", 5) == 0) { + str2 += 5; + proto_type = PROTO_TYPE_DGRAM; + ctrl_type = SOCK_STREAM; + } if (strncmp(str2, "unix@", 5) == 0) { str2 += 5;