Fix inconsistencies in hostname configuration property for Kerberos auth

trinodb · Feb 4, 2019 · 157ad3a · 157ad3a
1 parent d5dfc99
commit 157ad3a
Show file tree

Hide file tree

Showing 3 changed files with 3 additions and 3 deletions.
diff --git a/presto-docs/src/main/sphinx/security/server.rst b/presto-docs/src/main/sphinx/security/server.rst
@@ -87,7 +87,7 @@ Kerberos authentication is configured in the coordinator node's
     http-server.authentication.type=KERBEROS
 
     http.server.authentication.krb5.service-name=presto
-    http.server.authentication.krb5.host-name=presto.prestosql.io
+    http.server.authentication.krb5.principal-hostname=presto.prestosql.io
     http.server.authentication.krb5.keytab=/etc/presto/presto.keytab
     http.authentication.krb5.config=/etc/krb5.conf
 

diff --git a/presto-main/src/main/java/io/prestosql/server/security/KerberosConfig.java b/presto-main/src/main/java/io/prestosql/server/security/KerberosConfig.java
@@ -71,7 +71,7 @@ public String getPrincipalHostname()
         return principalHostname;
     }
 
-    @Config("http.authentication.krb5.principal-hostname")
+    @Config("http.server.authentication.krb5.principal-hostname")
     public KerberosConfig setPrincipalHostname(String principalHostname)
     {
         this.principalHostname = principalHostname;

diff --git a/presto-main/src/test/java/io/prestosql/server/security/TestKerberosConfig.java b/presto-main/src/test/java/io/prestosql/server/security/TestKerberosConfig.java
@@ -39,7 +39,7 @@ public void testExplicitPropertyMappings()
                 .put("http.authentication.krb5.config", "/etc/krb5.conf")
                 .put("http.server.authentication.krb5.service-name", "airlift")
                 .put("http.server.authentication.krb5.keytab", "/tmp/presto.keytab")
-                .put("http.authentication.krb5.principal-hostname", "presto.prestosql.io")
+                .put("http.server.authentication.krb5.principal-hostname", "presto.prestosql.io")
                 .build();
 
         KerberosConfig expected = new KerberosConfig()