[ https://issues.apache.org/jira/browse/HADOOP-10895?page=com.atlassian.jira.plugin.system.issuetabpanels:comment-tabpanel&focusedCommentId=14200489#comment-14200489 ]
Alejandro Abdelnur commented on HADOOP-10895: --------------------------------------------- The name of the properties and methods {{allowFallbackToPseudoAuth}} and {{allowDefaultAuthToFallbackToPseudo}} is a bit odd, I would have 'Default' at the end. In the {{KMSClientProvider}}, instead setting the default fallback, we should create the authenticator with the on/off setting and pass it to the authenticated URL. You don't want a global setting as some other place in the code could change the default and it will unknowingly impact the {{KMSCLientProvider}}. This should be the rule for all places where {{AuthenticatedURL}} is used. Other than that, the patch seem OK. Thx > HTTP KerberosAuthenticator fallback should have a flag to disable it > -------------------------------------------------------------------- > > Key: HADOOP-10895 > URL: https://issues.apache.org/jira/browse/HADOOP-10895 > Project: Hadoop Common > Issue Type: Bug > Components: security > Affects Versions: 2.4.1 > Reporter: Alejandro Abdelnur > Assignee: Yongjun Zhang > Priority: Blocker > Attachments: HADOOP-10895.001.patch, HADOOP-10895.002.patch, > HADOOP-10895.003.patch, HADOOP-10895.003v1.patch, HADOOP-10895.003v2.patch, > HADOOP-10895.003v2improved.patch, HADOOP-10895.004.patch, > HADOOP-10895.005.patch, HADOOP-10895.006.patch, HADOOP-10895.007.patch > > > Per review feedback in HADOOP-10771, {{KerberosAuthenticator}} and the > delegation token version coming in with HADOOP-10771 should have a flag to > disable fallback to pseudo, similarly to the one that was introduced in > Hadoop RPC client with HADOOP-9698. -- This message was sent by Atlassian JIRA (v6.3.4#6332)