You are viewing a plain text version of this content. The canonical link for it is here.
Posted to commits@kudu.apache.org by al...@apache.org on 2023/04/17 20:18:30 UTC

[kudu] branch master updated: Update security-itest jwt tests

This is an automated email from the ASF dual-hosted git repository.

alexey pushed a commit to branch master
in repository https://gitbox.apache.org/repos/asf/kudu.git


The following commit(s) were added to refs/heads/master by this push:
     new 08cb9b991 Update security-itest jwt tests
08cb9b991 is described below

commit 08cb9b99116a04ae7c4fa15d05286032983cee34
Author: Zoltan Chovan <zc...@cloudera.com>
AuthorDate: Mon Apr 17 18:31:38 2023 +0000

    Update security-itest jwt tests
    
    This change updates the 'TestJwtMiniClusterWithUntrustedCert' and
    'TestJwtMiniClusterWithInvalidCert' tests so that they are not relying
    on third party (curl) generated error messages, but only on messages
    coming from Kudu.
    
    Change-Id: Ie6727ec5c4fee6ad791b9ab5446f31ee2bf97c22
    Reviewed-on: http://gerrit.cloudera.org:8080/19758
    Reviewed-by: Alexey Serbin <al...@apache.org>
    Tested-by: Alexey Serbin <al...@apache.org>
---
 src/kudu/integration-tests/security-itest.cc | 7 ++++---
 1 file changed, 4 insertions(+), 3 deletions(-)

diff --git a/src/kudu/integration-tests/security-itest.cc b/src/kudu/integration-tests/security-itest.cc
index 0b77adbb9..8987292dd 100644
--- a/src/kudu/integration-tests/security-itest.cc
+++ b/src/kudu/integration-tests/security-itest.cc
@@ -655,8 +655,8 @@ TEST_F(SecurityITest, TestJwtMiniClusterWithInvalidCert) {
 
     Status s = client_builder.Build(&client);
     ASSERT_FALSE(s.ok());
-    ASSERT_STR_CONTAINS(s.ToString(),
-                        "SSL certificate problem: unable to get local issuer certificate");
+    ASSERT_TRUE(s.IsRuntimeError());
+    ASSERT_STR_CONTAINS(s.ToString(), "Error initializing JWT helper: Failed to load JWKS");
   }
 }
 
@@ -708,7 +708,8 @@ TEST_F(SecurityITest, TestJwtMiniClusterWithUntrustedCert) {
 
     Status s = client_builder.Build(&client);
     ASSERT_FALSE(s.ok());
-    ASSERT_STR_CONTAINS(s.ToString(), "SSL peer certificate or SSH remote key was not OK");
+    ASSERT_TRUE(s.IsRuntimeError());
+    ASSERT_STR_CONTAINS(s.ToString(), "Error initializing JWT helper: Failed to load JWKS");
   }
 }