From: Frederic Lecaille <flecaille@haproxy.com>
Date: Wed, 25 Feb 2026 10:25:05 +0000 (+0100)
Subject: CLEANUP: ssl: remove outdated comments
X-Git-Tag: v3.4-dev6~82
X-Git-Url: http://git.ipfire.org/gitweb.cgi?a=commitdiff_plain;h=5af42fa3420a87066739dfcfd49afbfc528550dc;p=thirdparty%2Fhaproxy.git

CLEANUP: ssl: remove outdated comments

ssl_sock_srv_try_reuse_sess() was modified by this commit to no longer
fail (it now returns void), but the related comments remained:

  BUG/MINOR: quic: missing app ops init during backend 0-RTT sessions

This patch cleans them up.
---

diff --git a/src/ssl_sock.c b/src/ssl_sock.c
index 19b975283..420894a08 100644
--- a/src/ssl_sock.c
+++ b/src/ssl_sock.c
@@ -5686,26 +5686,17 @@ int increment_sslconn()
 
 /* Try to reuse an SSL session (SSL_SESSION object) for <srv> server with <ctx>
  * as SSL socket context.
- * Return 1 if succeeded, 0 if not. Always succeeds for TCP socket. May fail
- * for QUIC sockets.
  */
 void ssl_sock_srv_try_reuse_sess(struct ssl_sock_ctx *ctx, struct server *srv)
 {
 #ifdef USE_QUIC
 	struct quic_conn *qc = ctx->qc;
-	/* Default status for QUIC sockets + 0-RTT is failure(0). The status will
-	 * be set to success(1) only if the QUIC connection parameters
-	 * (transport parameters and ALPN) are successfully reused.
-	 */
 	struct connection *conn = qc ? qc->conn : ctx->conn;
 #else
-	/* Always succeeds for TCP sockets. */
 	struct connection *conn = ctx->conn;
 #endif
 
-	/*
-	 * Always fail for check connections
-	 */
+	/* Do nothing for check connections */
 	if (conn->flags & CO_FL_SSL_NO_CACHED_INFO)
 		return;