Wenzhe Zhou has posted comments on this change. ( 
http://gerrit.cloudera.org:8080/18470 )

Change subject: server: link in kudu_jwt_util
......................................................................


Patch Set 9:

(1 comment)

http://gerrit.cloudera.org:8080/#/c/18470/4/src/kudu/server/server_base.cc
File src/kudu/server/server_base.cc:

http://gerrit.cloudera.org:8080/#/c/18470/4/src/kudu/server/server_base.cc@661
PS4, Line 661: LAGS_keytab_file)
> JwtVerifier is base class. We need to use KeyBasedJwtVerifier
How about std::make_shared<KeyBasedJwtVerifier>("", true)?



--
To view, visit http://gerrit.cloudera.org:8080/18470
To unsubscribe, visit http://gerrit.cloudera.org:8080/settings

Gerrit-Project: kudu
Gerrit-Branch: master
Gerrit-MessageType: comment
Gerrit-Change-Id: Icfe694d553ebead6afbf58dc773bf5534f1d099a
Gerrit-Change-Number: 18470
Gerrit-PatchSet: 9
Gerrit-Owner: Andrew Wong <anjuw...@g.ucla.edu>
Gerrit-Reviewer: Alexey Serbin <ale...@apache.org>
Gerrit-Reviewer: Attila Bukor <abu...@apache.org>
Gerrit-Reviewer: Kudu Jenkins (120)
Gerrit-Reviewer: Marton Greber <greber...@gmail.com>
Gerrit-Reviewer: Wenzhe Zhou <wz...@cloudera.com>
Gerrit-Reviewer: Zoltan Chovan <zcho...@cloudera.com>
Gerrit-Comment-Date: Fri, 09 Dec 2022 18:47:56 +0000
Gerrit-HasComments: Yes

Reply via email to