Github user Ethanlm commented on a diff in the pull request:

    https://github.com/apache/storm/pull/2531#discussion_r164884692
  
    --- Diff: 
storm-client/src/jvm/org/apache/storm/security/auth/kerberos/KerberosSaslTransportPlugin.java
 ---
    @@ -194,15 +223,15 @@ public TTransport connect(TTransport transport, 
String serverHost, String asUser
             final Subject subject = login.getSubject();
             if (subject.getPrivateCredentials(KerberosTicket.class).isEmpty()) 
{ //error
                 throw new RuntimeException("Fail to verify user principal with 
section \""
    -                        +AuthUtils.LOGIN_CONTEXT_CLIENT+"\" in login 
configuration file "+ login_conf);
    +                        +AuthUtils.LOGIN_CONTEXT_CLIENT+"\" in login 
configuration file "+ loginConf);
    --- End diff --
    
    spacing


---

Reply via email to