[
https://issues.apache.org/jira/browse/DRILL-4280?page=com.atlassian.jira.plugin.system.issuetabpanels:comment-tabpanel&focusedCommentId=15803151#comment-15803151
]
ASF GitHub Bot commented on DRILL-4280:
---------------------------------------
Github user sohami commented on a diff in the pull request:
https://github.com/apache/drill/pull/578#discussion_r94864855
--- Diff:
exec/java-exec/src/test/java/org/apache/drill/exec/rpc/user/security/TestCustomUserAuthenticator.java
---
@@ -56,16 +68,27 @@ public void positiveUserAuth() throws Exception {
runTest(TEST_USER_2, TEST_USER_2_PASSWORD);
}
-
@Test
public void negativeUserAuth() throws Exception {
negativeAuthHelper(TEST_USER_1, "blah.. blah..");
negativeAuthHelper(TEST_USER_2, "blah.. blah..");
- negativeAuthHelper(TEST_USER_2, "");
negativeAuthHelper("invalidUserName", "blah.. blah..");
}
@Test
+ public void emptyPassword() throws Exception {
+ try {
+ runTest(TEST_USER_2, "");
+ fail("Expected an exception.");
+ } catch (RpcException e) {
+ final String exMsg = e.getMessage();
+ assertThat(exMsg, containsString("Insufficient credentials"));
--- End diff --
We should check the "cause" rather than message string since that can
change over time ? Same in negativeAuthHelper.
> Kerberos Authentication
> -----------------------
>
> Key: DRILL-4280
> URL: https://issues.apache.org/jira/browse/DRILL-4280
> Project: Apache Drill
> Issue Type: Improvement
> Reporter: Keys Botzum
> Assignee: Sudheesh Katkam
> Labels: security
>
> Drill should support Kerberos based authentication from clients. This means
> that both the ODBC and JDBC drivers as well as the web/REST interfaces should
> support inbound Kerberos. For Web this would most likely be SPNEGO while for
> ODBC and JDBC this will be more generic Kerberos.
> Since Hive and much of Hadoop supports Kerberos there is a potential for a
> lot of reuse of ideas if not implementation.
> Note that this is related to but not the same as
> https://issues.apache.org/jira/browse/DRILL-3584
--
This message was sent by Atlassian JIRA
(v6.3.4#6332)
