[PATCH] Tests: allow to run ssl_curve.t on BoringSSL

Sergey Kandaurov pluknet at nginx.com
Wed Aug 23 12:26:15 UTC 2023


# HG changeset patch
# User Sergey Kandaurov <pluknet at nginx.com>
# Date 1692793549 -14400
#      Wed Aug 23 16:25:49 2023 +0400
# Node ID fae298d5f8d649af8006707b2f2856cc07f220b9
# Parent  4d13c9e74d04bb6cdd83923ded045d665adc5226
Tests: allow to run ssl_curve.t on BoringSSL.

Recently BoringSSL has got SSL_get_negotiated_group(),
which makes $ssl_curve to return the expected value.

While here, moved SSL library check to runtime.

diff --git a/ssl_curve.t b/ssl_curve.t
--- a/ssl_curve.t
+++ b/ssl_curve.t
@@ -23,7 +23,7 @@ select STDERR; $| = 1;
 select STDOUT; $| = 1;
 
 my $t = Test::Nginx->new()
-	->has(qw/http http_ssl rewrite socket_ssl openssl:3.0.0/)
+	->has(qw/http http_ssl rewrite socket_ssl/)
 	->has_daemon('openssl');
 
 $t->write_file_expand('nginx.conf', <<'EOF');
@@ -75,6 +75,10 @@ foreach my $name ('localhost') {
 
 ###############################################################################
 
+local $TODO = 'OpenSSL too old'
+	unless $t->has_feature('openssl:3.0.0')
+	or $t->has_module('BoringSSL');
+
 like(http_get('/curve', SSL => 1), qr/^prime256v1 /m, 'ssl curve');
 
 ###############################################################################


More information about the nginx-devel mailing list