[jira] [Comment Edited] (SENTRY-615) [Unit Test] Fix TestSentryWebServerWithKerberos#testPingWithUnauthorizedUser failed in Jenkins occasionally

["Dapeng Sun (JIRA)" <ji...@apache.org>]

    [ https://issues.apache.org/jira/browse/SENTRY-615?page=com.atlassian.jira.plugin.system.issuetabpanels:comment-tabpanel&focusedCommentId=14288827#comment-14288827 ] 

Dapeng Sun edited comment on SENTRY-615 at 1/23/15 6:30 AM:
------------------------------------------------------------

You can find several code examples using log4j for testing, log4j provides rich features, it's a "better" System.out.println() for logging statements, We should consider to use it.
If you have any other question, please let me know.


was (Author: dapengsun):
You can use your IDE search LoggerFactory by yourself. Do you have any other question about this change?

> [Unit Test] Fix TestSentryWebServerWithKerberos#testPingWithUnauthorizedUser failed in Jenkins occasionally
> -----------------------------------------------------------------------------------------------------------
>
>                 Key: SENTRY-615
>                 URL: https://issues.apache.org/jira/browse/SENTRY-615
>             Project: Sentry
>          Issue Type: Bug
>            Reporter: Dapeng Sun
>            Assignee: Dapeng Sun
>         Attachments: SENTRY-615.001.patch, SENTRY-615.002.patch
>
>
> Since we use {{KerberosAuthenticator}} to test Sentry Web Authorization, it didn't handle the status code {{403}}, so it may throw Exception in two parts:[KerberosAuthenticator | https://github.com/apache/hadoop/blob/f71eb51ab8109c14e8e921751dd5de603bdf2bde/hadoop-common-project/hadoop-auth/src/main/java/org/apache/hadoop/security/authentication/client/KerberosAuthenticator.java#L345] and [AuthenticatedURL |https://github.com/apache/hadoop/blob/f71eb51ab8109c14e8e921751dd5de603bdf2bde/hadoop-common-project/hadoop-auth/src/main/java/org/apache/hadoop/security/authentication/client/AuthenticatedURL.java#L274], currently we only handle {{AuthenticatedURL}}, {{KerberosAuthenticator }} should also be considered



--
This message was sent by Atlassian JIRA
(v6.3.4#6332)