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

Chen Zhang edited comment on HDFS-14609 at 9/11/19 6:47 AM:
------------------------------------------------------------

Hi [~crh], thanks for your response, I've some explaination in this [comment 
|https://issues.apache.org/jira/browse/HDFS-14609?focusedCommentId=16907486&page=com.atlassian.jira.plugin.system.issuetabpanels%3Acomment-tabpanel#comment-16907486]about
 the change of test's behavior, FYI.

For your question, the answer is :
{quote} # For case {{testGetDelegationToken()}}, the server address is set by 
WebHdfsFileSystem after it get the response, the original address is the 
address of RouterRpcServer. Since we now send request by http connection 
directly, it's unnecessary to reset the address, so I removed this assert
 # For the case {{testCancelDelegationToken()}}, the {{InvalidToken}} exception 
is also generated by WebHdfsFileSystem and the logic is very complex, I think 
it's also unnecessary to keep this assert, I use the 403 detection 
instead.{quote}


was (Author: zhangchen):
Hi [~crh], thanks for your response, I've some explaination in this comment 
about the change of test's behavior, FYI.

For your question, the answer is :
{quote} # For case {{testGetDelegationToken()}}, the server address is set by 
WebHdfsFileSystem after it get the response, the original address is the 
address of RouterRpcServer. Since we now send request by http connection 
directly, it's unnecessary to reset the address, so I removed this assert
 # For the case {{testCancelDelegationToken()}}, the {{InvalidToken}} exception 
is also generated by WebHdfsFileSystem and the logic is very complex, I think 
it's also unnecessary to keep this assert, I use the 403 detection 
instead.{quote}

> RBF: Security should use common AuthenticationFilter
> ----------------------------------------------------
>
>                 Key: HDFS-14609
>                 URL: https://issues.apache.org/jira/browse/HDFS-14609
>             Project: Hadoop HDFS
>          Issue Type: Bug
>            Reporter: CR Hota
>            Assignee: Chen Zhang
>            Priority: Major
>         Attachments: HDFS-14609.001.patch, HDFS-14609.002.patch, 
> HDFS-14609.003.patch, HDFS-14609.004.patch, HDFS-14609.005.patch
>
>
> We worked on router based federation security as part of HDFS-13532. We kept 
> it compatible with the way namenode works. However with HADOOP-16314 and 
> HDFS-16354 in trunk, auth filters seems to have been changed causing tests to 
> fail.
> Changes are needed appropriately in RBF, mainly fixing broken tests.



--
This message was sent by Atlassian Jira
(v8.3.2#803003)

---------------------------------------------------------------------
To unsubscribe, e-mail: hdfs-issues-unsubscr...@hadoop.apache.org
For additional commands, e-mail: hdfs-issues-h...@hadoop.apache.org

Reply via email to