[ 
https://issues.apache.org/jira/browse/HADOOP-13228?page=com.atlassian.jira.plugin.system.issuetabpanels:all-tabpanel
 ]

Xiao Chen updated HADOOP-13228:
-------------------------------
    Attachment: HADOOP-13228.02.patch

Thanks again Andrew for the quick response.

Patch 2 fixes checkstyle, and adds a new dummy class to verify the DT on 
request header. (I meant to say inject :) )

I noticed TestWebDelegationToken can be cleaned up in various ways
- no test timeout
- every test method has {{final Server jetty = createJettyServer();}} at the 
beginning, and {{jetty.stop();}} in the end, which is exactly what the test's 
@Before and @After is doing...
- could use {{GenericTestUtil#assertExceptionContains}} for exception message 
assertion


But I want to keep this jira small and focused, so will file a new jira for 
test clean up if you agree. The new test I added is clean from the above.

> Add delegation token to the connection in DelegationTokenAuthenticator
> ----------------------------------------------------------------------
>
>                 Key: HADOOP-13228
>                 URL: https://issues.apache.org/jira/browse/HADOOP-13228
>             Project: Hadoop Common
>          Issue Type: Bug
>          Components: security
>    Affects Versions: 2.6.0
>            Reporter: Xiao Chen
>            Assignee: Xiao Chen
>         Attachments: HADOOP-13228.01.patch, HADOOP-13228.02.patch
>
>
> Following [a comment from another 
> jira|https://issues.apache.org/jira/browse/HADOOP-13155?focusedCommentId=15308715&page=com.atlassian.jira.plugin.system.issuetabpanels:comment-tabpanel#comment-15308715],
>  create this to specifically handle the delegation token renewal/cancellation 
> bug in {{DelegationTokenAuthenticatedURL}} and 
> {{DelegationTokenAuthenticator}}.



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

---------------------------------------------------------------------
To unsubscribe, e-mail: [email protected]
For additional commands, e-mail: [email protected]

Reply via email to