You are viewing a plain text version of this content. The canonical link for it is here.
Posted to bugs@httpd.apache.org by bu...@apache.org on 2020/08/31 06:19:47 UTC

[Bug 64699] New: Ab cannot support TLSv1.3

https://bz.apache.org/bugzilla/show_bug.cgi?id=64699

            Bug ID: 64699
           Summary: Ab cannot support TLSv1.3
           Product: Apache httpd-2
           Version: 2.5-HEAD
          Hardware: PC
            Status: NEW
          Severity: normal
          Priority: P2
         Component: mod_http2
          Assignee: bugs@httpd.apache.org
          Reporter: xiaolongx.jiang@intel.com
  Target Milestone: ---

Ab cannot support TLSv1.3.

TLSv1.3 is becoming more and more widely used, but AB is not currently
supported. Can you support it?

I made a small patch in httpd-2.4.46 to implement this function. Could you
please help me review and add this function.

See the attachment:
0001-support-TLS1.3.patch
Thank you!

-- 
You are receiving this mail because:
You are the assignee for the bug.
---------------------------------------------------------------------
To unsubscribe, e-mail: bugs-unsubscribe@httpd.apache.org
For additional commands, e-mail: bugs-help@httpd.apache.org


[Bug 64699] Ab cannot support TLSv1.3

Posted by bu...@apache.org.
https://bz.apache.org/bugzilla/show_bug.cgi?id=64699

Christophe JAILLET <ch...@wanadoo.fr> changed:

           What    |Removed                     |Added
----------------------------------------------------------------------------
             Status|NEW                         |NEEDINFO

--- Comment #1 from Christophe JAILLET <ch...@wanadoo.fr> ---
Hi jiangxiaolong,

no file has been attached so far to your report.

We are always pleased to receive new contribution (and new contributors). So
don't hesitate to come back to us.

Regards,
CJ

-- 
You are receiving this mail because:
You are the assignee for the bug.
---------------------------------------------------------------------
To unsubscribe, e-mail: bugs-unsubscribe@httpd.apache.org
For additional commands, e-mail: bugs-help@httpd.apache.org


[Bug 64699] Ab cannot support TLSv1.3

Posted by bu...@apache.org.
https://bz.apache.org/bugzilla/show_bug.cgi?id=64699

jiangxiaolong <xi...@intel.com> changed:

           What    |Removed                     |Added
----------------------------------------------------------------------------
            Version|2.5-HEAD                    |2.4.46
                 OS|                            |All

-- 
You are receiving this mail because:
You are the assignee for the bug.
---------------------------------------------------------------------
To unsubscribe, e-mail: bugs-unsubscribe@httpd.apache.org
For additional commands, e-mail: bugs-help@httpd.apache.org


[Bug 64699] Ab cannot support TLSv1.3

Posted by bu...@apache.org.
https://bz.apache.org/bugzilla/show_bug.cgi?id=64699

--- Comment #3 from jiangxiaolong <xi...@intel.com> ---
Created attachment 37414
  --> https://bz.apache.org/bugzilla/attachment.cgi?id=37414&action=edit
add support TLSv1.3 for ab

httpd-2.4.46, add support TLSv1.3 for ab.

-- 
You are receiving this mail because:
You are the assignee for the bug.
---------------------------------------------------------------------
To unsubscribe, e-mail: bugs-unsubscribe@httpd.apache.org
For additional commands, e-mail: bugs-help@httpd.apache.org


[Bug 64699] Ab cannot support TLSv1.3

Posted by bu...@apache.org.
https://bz.apache.org/bugzilla/show_bug.cgi?id=64699

Christophe JAILLET <ch...@wanadoo.fr> changed:

           What    |Removed                     |Added
----------------------------------------------------------------------------
             Status|NEEDINFO                    |NEW

-- 
You are receiving this mail because:
You are the assignee for the bug.
---------------------------------------------------------------------
To unsubscribe, e-mail: bugs-unsubscribe@httpd.apache.org
For additional commands, e-mail: bugs-help@httpd.apache.org


[Bug 64699] Ab cannot support TLSv1.3

Posted by bu...@apache.org.
https://bz.apache.org/bugzilla/show_bug.cgi?id=64699

--- Comment #2 from jiangxiaolong <xi...@intel.com> ---
From 8d043c184e103faa94d278eaa2f28db7699d6b1b Mon Sep 17 00:00:00 2001
From: Xiaolong Jiang <xi...@intel.com>
Date: Sun, 30 Aug 2020 02:14:44 +0000
Subject: [PATCH] ab support TLS1.3

Signed-off-by: Xiaolong Jiang <xi...@intel.com>
---
 support/ab.c | 9 ++++++---
 1 file changed, 6 insertions(+), 3 deletions(-)

diff --git a/support/ab.c b/support/ab.c
index 71cf484..3b1c699 100644
--- a/support/ab.c
+++ b/support/ab.c
@@ -2160,7 +2160,7 @@ static void usage(const char *progname)
 #endif

 #ifdef HAVE_TLSV1_X
-#define TLS1_X_HELP_MSG ", TLS1.1, TLS1.2"
+#define TLS1_X_HELP_MSG ", TLS1.1, TLS1.2, TLS1.3"
 #else
 #define TLS1_X_HELP_MSG ""
 #endif
@@ -2294,7 +2294,7 @@ int main(int argc, const char * const argv[])
     const char *opt_arg;
     char c;
 #if OPENSSL_VERSION_NUMBER >= 0x10100000L
-    int max_prot = TLS1_2_VERSION;
+    int max_prot = TLS1_3_VERSION;
 #ifndef OPENSSL_NO_SSL3
     int min_prot = SSL3_VERSION;
 #else
@@ -2559,7 +2559,7 @@ int main(int argc, const char * const argv[])
 #else /* #if OPENSSL_VERSION_NUMBER < 0x10100000L */
                 meth = TLS_client_method();
                 if (strncasecmp(opt_arg, "ALL", 3) == 0) {
-                    max_prot = TLS1_2_VERSION;
+                    max_prot = TLS1_3_VERSION;
 #ifndef OPENSSL_NO_SSL3
                     min_prot = SSL3_VERSION;
 #else
@@ -2576,6 +2576,9 @@ int main(int argc, const char * const argv[])
                 } else if (strncasecmp(opt_arg, "TLS1.2", 6) == 0) {
                     max_prot = TLS1_2_VERSION;
                     min_prot = TLS1_2_VERSION;
+                } else if (strncasecmp(opt_arg, "TLS1.3", 6) == 0) {
+                    max_prot = TLS1_3_VERSION;
+                    min_prot = TLS1_3_VERSION;
                 } else if (strncasecmp(opt_arg, "TLS1", 4) == 0) {
                     max_prot = TLS1_VERSION;
                     min_prot = TLS1_VERSION;
-- 
2.17.1

-- 
You are receiving this mail because:
You are the assignee for the bug.
---------------------------------------------------------------------
To unsubscribe, e-mail: bugs-unsubscribe@httpd.apache.org
For additional commands, e-mail: bugs-help@httpd.apache.org


[Bug 64699] Ab cannot support TLSv1.3

Posted by bu...@apache.org.
https://bz.apache.org/bugzilla/show_bug.cgi?id=64699

Christophe JAILLET <ch...@wanadoo.fr> changed:

           What    |Removed                     |Added
----------------------------------------------------------------------------
         Resolution|---                         |DUPLICATE
             Status|NEW                         |RESOLVED

--- Comment #4 from Christophe JAILLET <ch...@wanadoo.fr> ---
The patch proposed in bug 63594 is different, but the goal is the same.

*** This bug has been marked as a duplicate of bug 63594 ***

-- 
You are receiving this mail because:
You are the assignee for the bug.
---------------------------------------------------------------------
To unsubscribe, e-mail: bugs-unsubscribe@httpd.apache.org
For additional commands, e-mail: bugs-help@httpd.apache.org