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

Yongjun Zhang commented on HADOOP-10895:
----------------------------------------

Hi [~tucu00],

The concern you raised in your comment #2  is that some code might mess with 
the default fallback setting in the KerberosAuthenticator and cause unwanted 
effect.  I came up an alternative solution that I think would address this 
concern without having to do the change described in my last comment. I just 
uploaded rev 008, with your comment #1 addressed too.

That is, let KMSClientProvider remember the value of allowFallback specified in 
the configuration file as a private boolean member when KMSClientProvider 
object is constructed, and then refresh the KerberosAuthenticator's default 
setting each time before KMSClientProvider object creates AuthenticatatedURL 
object.

After all, our intention is that the default setting should be the same as 
specified in the configuration file all the time after initialization. What we 
are adding here is a protection in case some code accidentally changed the 
setting.

BTW, as far as I can see, the only production code that has the need for this 
change is KMSClientProvider, other similar places are in testing code. I think 
it's ok for the other places to rely on setting the default fallback at 
initialization time without refreshing, which would even help us to find any 
culprit code that tries to mess with the default setting after initialization, 
if error happens.

Would you please help take a look at rev 008?

Thanks a lot.


> 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, 
> HADOOP-10895.008.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)

Reply via email to