]> git.ipfire.org Git - thirdparty/squid.git/commitdiff
Bug 5093: List http_port params that https_port/ftp_port lack (#1977)
authorFrancesco Chemolli <5175948+kinkie@users.noreply.github.com>
Tue, 7 Jan 2025 23:20:48 +0000 (23:20 +0000)
committerFrancesco Chemolli <5175948+kinkie@users.noreply.github.com>
Sun, 19 Jan 2025 22:24:18 +0000 (22:24 +0000)
To avoid documentation duplication, current https_port and ftp_port
directive descriptions reference http_port directive instead of
detailing their own supported parameters. For https_port, this solution
creates a false impression that the directive supports all http_port
options. Our ftp_port documentation is better but still leaves the
reader guessing which options are actually supported.

This change starts enumerating http_port configuration parameters that
ftp_port and https_port directives do _not_ support. Eventually, Squid
should reject configurations with unsupported listening port options.

src/cf.data.pre

index b02def6e9d7d5eb24270a0042d87e12a650dab6a..11785b7ab00126bd0f662e737abd009794fabbd8 100644 (file)
@@ -2584,6 +2584,9 @@ DOC_START
        The tls-cert= option is mandatory on HTTPS ports.
 
        See http_port for a list of modes and options.
+       Not all http_port options are available for https_port.
+       Among the unavalable options:
+       - require-proxy-header
 DOC_END
 
 NAME: ftp_port
@@ -2648,6 +2651,9 @@ DOC_START
 
        Other http_port modes and options that are not specific to HTTP and
        HTTPS may also work.
+       Among the options that are not available for ftp_port:
+       - require-proxy-header
+       - ssl-bump
 DOC_END
 
 NAME: tcp_outgoing_tos tcp_outgoing_ds tcp_outgoing_dscp