Jason Fehr has posted comments on this change. ( http://gerrit.cloudera.org:8080/21382 )
Change subject: IMPALA-12559: Support x5c Parameter for RSA JSON Web Keys ...................................................................... Patch Set 22: Code-Review+1 (4 comments) Overall looks great. One small comment is still outstanding if there is time to address it: https://gerrit.cloudera.org/c/21382/20..22/be/src/util/jwt-util-test.cc#1267 http://gerrit.cloudera.org:8080/#/c/21382/20/be/src/util/jwt-util-test.cc File be/src/util/jwt-util-test.cc: http://gerrit.cloudera.org:8080/#/c/21382/20/be/src/util/jwt-util-test.cc@460 PS20, Line 460: kid_x5c) > Done Done http://gerrit.cloudera.org:8080/#/c/21382/20/be/src/util/jwt-util.cc File be/src/util/jwt-util.cc: http://gerrit.cloudera.org:8080/#/c/21382/20/be/src/util/jwt-util.cc@157 PS20, Line 157: if (!values[0].empty()) { > Added. Thanks. Done http://gerrit.cloudera.org:8080/#/c/21382/20/be/src/util/jwt-util.cc@395 PS20, Line 395: algorithm = "rs384"; > Added. Thanks. Done http://gerrit.cloudera.org:8080/#/c/21382/20/testdata/jwt/jwks_x5c_rs256.json File testdata/jwt/jwks_x5c_rs256.json: http://gerrit.cloudera.org:8080/#/c/21382/20/testdata/jwt/jwks_x5c_rs256.json@10 PS20, Line 10: "MIIE2jCCAsICAQ > The double quote is not shown in the UI of gerrit. Thanks for the follow-up. -- To view, visit http://gerrit.cloudera.org:8080/21382 To unsubscribe, visit http://gerrit.cloudera.org:8080/settings Gerrit-Project: Impala-ASF Gerrit-Branch: master Gerrit-MessageType: comment Gerrit-Change-Id: I70be6f9f54190544aa005b2644e2ed8db6f6bb74 Gerrit-Change-Number: 21382 Gerrit-PatchSet: 22 Gerrit-Owner: gaurav singh <[email protected]> Gerrit-Reviewer: Abhishek Rawat <[email protected]> Gerrit-Reviewer: Impala Public Jenkins <[email protected]> Gerrit-Reviewer: Jason Fehr <[email protected]> Gerrit-Reviewer: Wenzhe Zhou <[email protected]> Gerrit-Reviewer: gaurav singh <[email protected]> Gerrit-Comment-Date: Tue, 14 May 2024 16:16:16 +0000 Gerrit-HasComments: Yes
