[ https://issues.apache.org/jira/browse/HBASE-19093?page=com.atlassian.jira.plugin.system.issuetabpanels:comment-tabpanel&focusedCommentId=16272940#comment-16272940 ]
Chia-Ping Tsai commented on HBASE-19093: ---------------------------------------- Thanks [~balazs.meszaros] for doing the garden works. Could you also list the excluded methods? That help us do the double-check. bq. What do you think, all of these method should have AccessController hooks? As I see it, all of them should have hooks and security checks unless it will cause the harmful performance regression. As [~anoop.hbase] suggested, lets open sub task for each interface? > Check Admin/Table to ensure all operations go via AccessControl > --------------------------------------------------------------- > > Key: HBASE-19093 > URL: https://issues.apache.org/jira/browse/HBASE-19093 > Project: HBase > Issue Type: Sub-task > Reporter: stack > Assignee: Balazs Meszaros > Priority: Blocker > Fix For: 2.0.0-beta-1 > > Attachments: HBASE-19093.master.001.patch, > HBASE-19093.master.002.patch, RegionObserver.txt > > > A cursory review of Admin Interface has a bunch of methods as open, with out > AccessControl checks. For example, procedure executor has not check on it. > This issue is about given the Admin and Table Interfaces a once-over to see > what is missing and to fill in access control where missing. > This is a follow-on from work over in HBASE-19048 -- This message was sent by Atlassian JIRA (v6.4.14#64029)