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

Duo Zhang commented on HBASE-19634:
-----------------------------------

There are permission checks for open/close region at RS side, which requires 
the user to have global ADMIN permission.

I think we should also require this permission for the executeProcedure call. 
So we need to add two methods in RegionServerObserver.

> Confirm that we do not leak the privilege for modifying replication peer to 
> unauthorized user
> ---------------------------------------------------------------------------------------------
>
>                 Key: HBASE-19634
>                 URL: https://issues.apache.org/jira/browse/HBASE-19634
>             Project: HBase
>          Issue Type: Sub-task
>          Components: proc-v2, Replication
>            Reporter: Duo Zhang
>
> This is important, the actual refresh on RS is trigger by the 
> executeProcedure call and it will pass some information. These information 
> should not be fully trusted since anyone can all this method. We need to make 
> sure that the actual data/state for a replication peer is always loaded from 
> the replication storage, not from the parameter of the executeProcedure call.



--
This message was sent by Atlassian JIRA
(v6.4.14#64029)

Reply via email to