Fix incorrect log formatting in DruidKerberosAuthenticationHandler. (a…

…pache#4817)
vogali · Sep 18, 2017 · 96612cc · 96612cc
1 parent c2a0e75
commit 96612cc
Showing 1 changed file with 3 additions and 5 deletions.
diff --git a/...kerberos/src/main/java/io/druid/security/kerberos/DruidKerberosAuthenticationHandler.java b/...kerberos/src/main/java/io/druid/security/kerberos/DruidKerberosAuthenticationHandler.java
@@ -106,9 +106,7 @@ public void init(Properties config) throws ServletException
       }
 
       for (String spnegoPrincipal : spnegoPrincipals) {
-        log.info("Login using keytab {}, for principal {}",
-                 keytab, spnegoPrincipal
-        );
+        log.info("Login using keytab %s, for principal %s", keytab, spnegoPrincipal);
         final KerberosAuthenticator.DruidKerberosConfiguration kerberosConfiguration =
             new KerberosAuthenticator.DruidKerberosConfiguration(keytab, spnegoPrincipal);
         final LoginContext loginContext =
@@ -117,7 +115,7 @@ public void init(Properties config) throws ServletException
           loginContext.login();
         }
         catch (LoginException le) {
-          log.warn(le, "Failed to login as [{}]", spnegoPrincipal);
+          log.warn(le, "Failed to login as [%s]", spnegoPrincipal);
           throw new AuthenticationException(le);
         }
         loginContexts.add(loginContext);
@@ -201,7 +199,7 @@ public AuthenticationToken run() throws Exception
                 String userName = kerberosName.getShortName();
                 token = new AuthenticationToken(userName, clientPrincipal, getType());
                 response.setStatus(HttpServletResponse.SC_OK);
-                log.trace("SPNEGO completed for principal [{}]", clientPrincipal);
+                log.trace("SPNEGO completed for principal [%s]", clientPrincipal);
               }
             }
             finally {