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

Anoop Sam John commented on HBASE-10883:
----------------------------------------

Why to add pattern and regex to HConstants?  
We better keep the validation at server side?  
And move it to VisibilityLabelsValidator?  
VisibilityLabelsValidator#isValidLabel(String)

> Restrict the universe of labels and authorizations
> --------------------------------------------------
>
>                 Key: HBASE-10883
>                 URL: https://issues.apache.org/jira/browse/HBASE-10883
>             Project: HBase
>          Issue Type: Improvement
>    Affects Versions: 0.98.1
>            Reporter: Andrew Purtell
>             Fix For: 0.99.0, 0.98.2
>
>         Attachments: HBASE-10883.patch, HBASE-10883_1.patch
>
>
> Currently we allow any string as visibility label or request authorization. 
> However as seen on HBASE-10878, we accept for authorizations strings that 
> would not work if provided as labels in visibility expressions. We should 
> throw an exception at least in cases where someone tries to define or use a 
> label or authorization including visibility expression operators '&', '|', 
> '!', '(', ')'.



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

Reply via email to