hbase-issues mailing list archives

Site index · List index
Message view « Date » · « Thread »
Top « Date » · « Thread »
From "Matteo Bertozzi (JIRA)" <j...@apache.org>
Subject [jira] [Commented] (HBASE-15200) ZooKeeper znode ACL checks should only compare the shortname
Date Thu, 04 Feb 2016 19:09:40 GMT

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

Matteo Bertozzi commented on HBASE-15200:
-----------------------------------------

looks like the on-commit fix, was done the other way around:
In the patch posted we just needed to fix the one String.format() at the end of the patch
that has %0x instead of 0x%x
post commit the correct 0x%x got changed in the wrong %0x which is causing findbugs reports
in other patches

> ZooKeeper znode ACL checks should only compare the shortname
> ------------------------------------------------------------
>
>                 Key: HBASE-15200
>                 URL: https://issues.apache.org/jira/browse/HBASE-15200
>             Project: HBase
>          Issue Type: Bug
>    Affects Versions: 2.0.0, 1.2.0, 1.0.3, 1.1.3, 0.98.17
>            Reporter: Andrew Purtell
>            Assignee: Andrew Purtell
>            Priority: Minor
>             Fix For: 2.0.0, 1.3.0, 1.2.1, 1.1.4, 1.0.4, 0.98.18
>
>         Attachments: HBASE-15200-branch-1.0.patch, HBASE-15200-branch-1.1.patch, HBASE-15200.patch,
HBASE-15200.patch
>
>
> After HBASE-13768 we check at startup in secure configurations if our znodes have the
correct ACLs. However when checking the ACL we compare the Kerberos fullname, which includes
the host component. We should only compare the shortname, the principal. Otherwise in a multimaster
configuration we will unnecessarily reset ACLs whenever any master running on a host other
than the one that initialized the ACLs makes the check. You can imagine this happening multiple
times in a rolling restart scenario.



--
This message was sent by Atlassian JIRA
(v6.3.4#6332)

Mime
View raw message