mirrors/haproxy
1
0
Fork 0
mirror of https://git.haproxy.org/git/haproxy.git/ synced 2025-08-06 15:17:01 +02:00
Code Issues Projects Releases Wiki Activity

MEDIUM: proxy: deprecate the "transparent" and "option transparent" directives

Browse Source 
As discussed here [1], "transparent" (already deprecated) and
"option transparent" are horrible hacks which should really disappear
in favor of "server xxx 0.0.0.0" which doesn't rely on hackish code
path. This old feature is now deprecated in 3.3 and will disappear in
3.5, as indicated here [2]. A warning is emitted when used, explaining
how to proceed, and how to silence the warning using the global
"expose-deprecated-directives" if needed. The doc was updated to
reflect this new state.

[1] https://github.com/orgs/haproxy/discussions/2921
[2] https://github.com/haproxy/wiki/wiki/Breaking-changes
This commit is contained in:
Willy Tarreau 2025-06-26 11:55:47 +02:00
parent 194e3bc2d5
commit e93f3ea3f8
2 changed files with 21 additions and 2 deletions
Show Stats Download Patch File Download Diff File Expand all files Collapse all files

2
doc/configuration.txt
View File

@ -5578,7 +5578,7 @@ option tcp-smart-accept (*) X X X -
option tcp-smart-connect (*) X - X X option tcp-smart-connect (*) X - X X
option tcpka X X X X option tcpka X X X X
option tcplog X X X - option tcplog X X X -
option transparent (*) X - X X option transparent (deprecated) (*) X - X X
option idle-close-on-response (*) X X X - option idle-close-on-response (*) X X X -
external-check command X - X X external-check command X - X X
external-check path X - X X external-check path X - X X

21
src/cfgparse-listen.c
View File

@ -2054,8 +2054,19 @@ int cfg_parse_listen(const char *file, int linenum, char **args, int kwm)
/* try to match option within cfg_opts */ /* try to match option within cfg_opts */
if (cfg_parse_listen_match_option(file, linenum, kwm, cfg_opts, &err_code, args, if (cfg_parse_listen_match_option(file, linenum, kwm, cfg_opts, &err_code, args,
PR_MODES, PR_CAP_NONE, PR_MODES, PR_CAP_NONE,
&curproxy->options, &curproxy->no_options)) &curproxy->options, &curproxy->no_options)) {
if (strcmp(args[1], "transparent") == 0) {
if (!deprecated_directives_allowed) {
ha_warning("parsing [%s:%d]: option '%s' is deprecated in 3.3 and will be removed in 3.5. "
"The modern way to do the same is to create a server with address 0.0.0.0. It is "
"still possible to silence this warning by setting 'expose-deprecated-directives' "
"in the 'global' section, but do not wait to fix your configuration!\n",
file, linenum, args[1]);
err_code |= ERR_WARN;
}
}
goto out; goto out;
}
if (err_code & ERR_CODE) if (err_code & ERR_CODE)
goto out; goto out;
@ -2574,6 +2585,14 @@ int cfg_parse_listen(const char *file, int linenum, char **args, int kwm)
curproxy->options |= PR_O_TRANSP; curproxy->options |= PR_O_TRANSP;
if (alertif_too_many_args(0, file, linenum, args, &err_code)) if (alertif_too_many_args(0, file, linenum, args, &err_code))
goto out; goto out;
if (!deprecated_directives_allowed) {
ha_warning("parsing [%s:%d]: '%s' is deprecated in 3.3 and will be removed in 3.5. "
"The modern way to do the same is to create a server with address 0.0.0.0. It is "
"still possible to silence this warning by setting 'expose-deprecated-directives' "
"in the 'global' section, but do not wait to fix your configuration!\n",
file, linenum, args[0]);
err_code |= ERR_WARN;
}
} }
#endif #endif
else if (strcmp(args[0], "maxconn") == 0) { /* maxconn */ else if (strcmp(args[0], "maxconn") == 0) { /* maxconn */