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

Daryn Sharp commented on HADOOP-10499:
--------------------------------------

Although it makes sense to remove an unused parameter, I wonder if the 
parameter should be honored?  It would allow different rpc services to have 
different proxy superuser configurations as opposed to the current static 
configuration...  It probably made sense when daemons generally had only one 
rpc service.  I don't have a strong opinion because I suppose it could be 
re-added/implemented by another jira.

> Remove unused parameter from ProxyUsers.authorize()
> ---------------------------------------------------
>
>                 Key: HADOOP-10499
>                 URL: https://issues.apache.org/jira/browse/HADOOP-10499
>             Project: Hadoop Common
>          Issue Type: Sub-task
>          Components: security
>    Affects Versions: 2.4.0
>            Reporter: Benoy Antony
>            Assignee: Benoy Antony
>            Priority: Minor
>         Attachments: HADOOP-10499.patch
>
>
> The Configuration parameter is not used in the authorize() function. It can 
> be removed and callers can be updated.
> Attaching the simple patch which removes the unused _conf_ parameter and 
> updates the callers.
> The ProxyUsers is defined as a private audience and so there shouldn't be any 
> external callers. 



--
This message was sent by Atlassian JIRA
(v6.2#6252)

Reply via email to