On Tue, Jan 18, 2022 at 12:07:21PM +0100, Willy Tarreau wrote: > > On Mon, Jan 17, 2022 at 07:46:24PM +0100, William Dauchy wrote: > > Hello Christopher, > > > > On Wed, Jan 12, 2022 at 12:45 PM William Dauchy <[email protected]> wrote: > > > my approach was to say: > > > - remove the implicit behavior > > > - then work on the missing commands for the health checks > > > > Do you think we can conclude on it? > > Just merged after our discussion on it :-) >
Can we also mark it as deprecated in 2.5? patch attached -- William Lallemand
>From 9998a33d3a027ff6863eab71bcc2f2d7158319b4 Mon Sep 17 00:00:00 2001 From: William Lallemand <[email protected]> Date: Wed, 19 Jan 2022 15:17:08 +0100 Subject: [PATCH] DOC: management: mark "set server ssl" as deprecated This command was integrated in 2.4 when it was not possible to handle SSL with dynamic servers, this is now possible so we should prefer this way. Must be backported in 2.5. --- doc/management.txt | 5 ++++- 1 file changed, 4 insertions(+), 1 deletion(-) diff --git a/doc/management.txt b/doc/management.txt index b96cbc8789..b9b7ebdfb7 100644 --- a/doc/management.txt +++ b/doc/management.txt @@ -2185,11 +2185,14 @@ set server <backend>/<server> fqdn <FQDN> Change a server's FQDN to the value passed in argument. This requires the internal run-time DNS resolver to be configured and enabled for this server. -set server <backend>/<server> ssl [ on | off ] +set server <backend>/<server> ssl [ on | off ] (deprecated) This option configures SSL ciphering on outgoing connections to the server. When switch off, all traffic becomes plain text; health check path is not changed. + This command is deprecated, create a new server dynamically with or without + SSL instead, using the "add server" command. + set severity-output [ none | number | string ] Change the severity output format of the stats socket connected to for the duration of the current session. -- 2.32.0
