acelyc111 commented on code in PR #1569:
URL: 
https://github.com/apache/incubator-pegasus/pull/1569#discussion_r1290846551


##########
src/runtime/security/init.cpp:
##########
@@ -43,6 +46,12 @@ void set_krb5_env(bool is_server)
 
 error_s init_kerberos(bool is_server)
 {
+    // When pegasus enable auth and lacks the necessary parameters to execute 
kinit by itself, and
+    // will try to obtain the principal under the current unix account for 
identity authentication.

Review Comment:
   ```suggestion
       // When FLAGS_enable_auth is set but lacks of necessary parameters to 
execute kinit by itself, then
       // try to obtain the principal under the current Unix account for 
identity authentication
       // automatically.
   ```



-- 
This is an automated message from the Apache Git Service.
To respond to the message, please log on to GitHub and use the
URL above to go to the specific comment.

To unsubscribe, e-mail: [email protected]

For queries about this service, please contact Infrastructure at:
[email protected]


---------------------------------------------------------------------
To unsubscribe, e-mail: [email protected]
For additional commands, e-mail: [email protected]

Reply via email to