[ https://issues.apache.org/jira/browse/HDFS-14609?page=com.atlassian.jira.plugin.system.issuetabpanels:comment-tabpanel&focusedCommentId=16924776#comment-16924776 ]
Chen Zhang edited comment on HDFS-14609 at 9/7/19 8:42 AM: ----------------------------------------------------------- Thanks [~elgoiri] for your review and comments. {quote}TestRouterWithSecureStartup#HTTP_KERBEROS_PRINCIPAL_CONF_KEY, don't we have this defined somewhere else?{quote} The conf key is joined by the {{HttpServer2#authFilterConfigurationPrefix}} and ".type" string, so I thought that there might not be a complete key already defined in somewhere else. But I'm wrong, {{CommonConfigurationKeysPublic#HADOOP_HTTP_AUTHENTICATION_TYPE}} defines it. I'll update the patch to use it instead. {quote}BTW, I've seen the use of equivalents to NoAuthFilterInitializer, don't we have this already in the common test lib?{quote}Can you point out where is the equivalents of NoAuthFilterInitializer? I try to find it by going through all the sub class of {{FilterInitializer}}, but not found. was (Author: zhangchen): Thanks [~elgoiri] for your review and comments. {quote}TestRouterWithSecureStartup#HTTP_KERBEROS_PRINCIPAL_CONF_KEY, don't we have this defined somewhere else?{quote} The conf key is joined by the {{HttpServer2#authFilterConfigurationPrefix}} and ".type" string, so I thought that there might not be a complete key already defined in somewhere else. But I'm wrong, {{CommonConfigurationKeysPublic#HADOOP_HTTP_AUTHENTICATION_TYPE}} defines it. I'll update the patch to use this instead. {quote}BTW, I've seen the use of equivalents to NoAuthFilterInitializer, don't we have this already in the common test lib?{quote}Can you point out where is the equivalents of NoAuthFilterInitializer? I try to find it by going through all the sub class of {{FilterInitializer}}, but not found. > 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 > > > 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