CLEANUP: muxes: Remove MX_FL_CLEAN_ABRT flag

This flag is unused. Thus, it may be removed. No reason to still set it. It
also cleans up "haproxy -vv" output.

diff --git a/doc/configuration.txt b/doc/configuration.txt
index 1953a2bb0e0c098e831fd1bbf6fdfd60c0914b96..361f9ee79f2bbde89019c7884183446d128d1e42 100644 (file)
--- a/doc/configuration.txt
+++ b/doc/configuration.txt
@@ -14102,15 +14102,14 @@ proto <name>
   in haproxy -vv. The protocols properties are reported : the mode (TCP/HTTP),
   the side (FE/BE), the mux name and its flags.
 
-  Some protocols report errors on aborts (flag=CLEAN_ABRT). Some others are
-  subject to the head-of-line blocking on server side (flag=HOL_RISK). Finally
-  some protocols don't support upgrades (flag=NO_UPG). The HTX compatibility is
-  also reported (flag=HTX).
+  Some protocols are subject to the head-of-line blocking on server side
+  (flag=HOL_RISK). Finally some protocols don't support upgrades (flag=NO_UPG).
+  The HTX compatibility is also reported (flag=HTX).
 
   Here are the protocols that may be used as argument to a "proto" directive on
   a bind line :
 
-    h2   : mode=HTTP  side=FE|BE  mux=H2    flags=HTX|CLEAN_ABRT|HOL_RISK|NO_UPG
+    h2   : mode=HTTP  side=FE|BE  mux=H2    flags=HTX|HOL_RISK|NO_UPG
     h1   : mode=HTTP  side=FE|BE  mux=H1    flags=HTX|NO_UPG
     none : mode=TCP   side=FE|BE  mux=PASS  flags=NO_UPG
 
@@ -14526,15 +14525,14 @@ check-proto <name>
   protocols is reported in haproxy -vv. The protocols properties are
   reported : the mode (TCP/HTTP), the side (FE/BE), the mux name and its flags.
 
-  Some protocols report errors on aborts (flag=CLEAN_ABRT). Some others are
-  subject to the head-of-line blocking on server side (flag=HOL_RISK). Finally
-  some protocols don't support upgrades (flag=NO_UPG). The HTX compatibility is
-  also reported (flag=HTX).
+  Some protocols are subject to the head-of-line blocking on server side
+  (flag=HOL_RISK). Finally some protocols don't support upgrades (flag=NO_UPG).
+  The HTX compatibility is also reported (flag=HTX).
 
   Here are the protocols that may be used as argument to a "check-proto"
   directive on a server line:
 
-    h2   : mode=HTTP  side=FE|BE  mux=H2    flags=HTX|CLEAN_ABRT|HOL_RISK|NO_UPG
+    h2   : mode=HTTP  side=FE|BE  mux=H2    flags=HTX|HOL_RISK|NO_UPG
     fcgi : mode=HTTP  side=BE     mux=FCGI  flags=HTX|HOL_RISK|NO_UPG
     h1   : mode=HTTP  side=FE|BE  mux=H1    flags=HTX|NO_UPG
     none : mode=TCP   side=FE|BE  mux=PASS  flags=NO_UPG
@@ -15038,15 +15036,14 @@ proto <name>
   reported in haproxy -vv.The protocols properties are reported : the mode
   (TCP/HTTP), the side (FE/BE), the mux name and its flags.
 
-  Some protocols report errors on aborts (flag=CLEAN_ABRT). Some others are
-  subject to the head-of-line blocking on server side (flag=HOL_RISK). Finally
-  some protocols don't support upgrades (flag=NO_UPG). The HTX compatibility is
-  also reported (flag=HTX).
+  Some protocols are subject to the head-of-line blocking on server side
+  (flag=HOL_RISK). Finally some protocols don't support upgrades (flag=NO_UPG).
+  The HTX compatibility is also reported (flag=HTX).
 
   Here are the protocols that may be used as argument to a "proto" directive on
   a server line :
 
-    h2   : mode=HTTP  side=FE|BE  mux=H2    flags=HTX|CLEAN_ABRT|HOL_RISK|NO_UPG
+    h2   : mode=HTTP  side=FE|BE  mux=H2    flags=HTX|HOL_RISK|NO_UPG
     fcgi : mode=HTTP  side=BE     mux=FCGI  flags=HTX|HOL_RISK|NO_UPG
     h1   : mode=HTTP  side=FE|BE  mux=H1    flags=HTX|NO_UPG
     none : mode=TCP   side=FE|BE  mux=PASS  flags=NO_UPG

diff --git a/include/haproxy/connection-t.h b/include/haproxy/connection-t.h
index 773610803d712df8d035a0a1a1c7c6a7d42c752a..22160f3f30dd14e223651ecf7a5daeef12c0a724 100644 (file)
--- a/include/haproxy/connection-t.h
+++ b/include/haproxy/connection-t.h
@@ -276,10 +276,9 @@ enum {
 /* MUX-specific flags */
 enum {
        MX_FL_NONE        = 0x00000000,
-       MX_FL_CLEAN_ABRT  = 0x00000001, /* abort is clearly reported as an error */
-       MX_FL_HTX         = 0x00000002, /* set if it is an HTX multiplexer */
-       MX_FL_HOL_RISK    = 0x00000004, /* set if the protocol is subject the to head-of-line blocking on server */
-       MX_FL_NO_UPG      = 0x00000008, /* set if mux does not support any upgrade */
+       MX_FL_HTX         = 0x00000001, /* set if it is an HTX multiplexer */
+       MX_FL_HOL_RISK    = 0x00000002, /* set if the protocol is subject the to head-of-line blocking on server */
+       MX_FL_NO_UPG      = 0x00000004, /* set if mux does not support any upgrade */
 };
 
 /* PROTO token registration */

diff --git a/src/connection.c b/src/connection.c
index dbfc0ea3bfcfcc211481518e2632c8771b98db47..88e851c93256b40f00e61db4997d49abd3822089 100644 (file)
--- a/src/connection.c
+++ b/src/connection.c
@@ -1716,9 +1716,6 @@ void list_mux_proto(FILE *out)
                if (item->mux->flags & MX_FL_HTX)
                        done |= fprintf(out, "%sHTX", done ? "|" : "");
 
-               if (item->mux->flags & MX_FL_CLEAN_ABRT)
-                       done |= fprintf(out, "%sCLEAN_ABRT", done ? "|" : "");
-
                if (item->mux->flags & MX_FL_HOL_RISK)
                        done |= fprintf(out, "%sHOL_RISK", done ? "|" : "");
 

diff --git a/src/mux_h2.c b/src/mux_h2.c
index e42cd8928bf4324ac25310c88a58f8f6186fdca9..71327b86b30a78ee9b71ae36e2e0af5ac865c757 100644 (file)
--- a/src/mux_h2.c
+++ b/src/mux_h2.c
@@ -6911,7 +6911,7 @@ static const struct mux_ops h2_ops = {
        .ctl = h2_ctl,
        .show_fd = h2_show_fd,
        .takeover = h2_takeover,
-       .flags = MX_FL_CLEAN_ABRT|MX_FL_HTX|MX_FL_HOL_RISK|MX_FL_NO_UPG,
+       .flags = MX_FL_HTX|MX_FL_HOL_RISK|MX_FL_NO_UPG,
        .name = "H2",
 };
 

diff --git a/src/mux_quic.c b/src/mux_quic.c
index e207b3aef1d30edada90fa048f1e9756873295a9..2051bff105e79a7fd524bbf8cd45bb159d0b9aed 100644 (file)
--- a/src/mux_quic.c
+++ b/src/mux_quic.c
@@ -1361,7 +1361,7 @@ static const struct mux_ops qc_ops = {
        .subscribe = qc_subscribe,
        .unsubscribe = qc_unsubscribe,
        .wake = qc_wake,
-       .flags = MX_FL_CLEAN_ABRT|MX_FL_HTX|MX_FL_NO_UPG,
+       .flags = MX_FL_HTX|MX_FL_NO_UPG,
        .name = "QUIC",
 };