MINOR: config: mark global.debug as deprecated

This directive has never made any sense and has already caused trouble
by forcing the process to stay in foreground during the boot process.
Let's emit a warning mentioning it's deprecated and will be removed in
2.3.
This commit is contained in:
Willy Tarreau 2020-02-25 11:27:22 +01:00
parent 41136de58e
commit 1b85785bc2
2 changed files with 4 additions and 1 deletions

View File

@ -2159,7 +2159,7 @@ tune.zlib.windowsize <number>
3.3. Debugging 3.3. Debugging
-------------- --------------
debug debug (deprecated)
Enables debug mode which dumps to stdout all exchanges, and disables forking Enables debug mode which dumps to stdout all exchanges, and disables forking
into background. It is the equivalent of the command-line argument "-d". It into background. It is the equivalent of the command-line argument "-d". It
should never be used in a production configuration since it may prevent full should never be used in a production configuration since it may prevent full

View File

@ -57,6 +57,9 @@ int cfg_parse_global(const char *file, int linenum, char **args, int kwm)
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;
global.mode |= MODE_DEBUG; global.mode |= MODE_DEBUG;
ha_warning("parsing [%s:%d] : '%s' must never be used and will be removed in 2.3. If you need debug mode, please use '-d' on the command line.\n", file, linenum, args[0]);
err_code |= ERR_WARN;
goto out;
} }
else if (!strcmp(args[0], "noepoll")) { else if (!strcmp(args[0], "noepoll")) {
if (alertif_too_many_args(0, file, linenum, args, &err_code)) if (alertif_too_many_args(0, file, linenum, args, &err_code))