curator-dev mailing list archives

Site index · List index
Message view « Date » · « Thread »
Top « Date » · « Thread »
From "Robert Kanter (JIRA)" <j...@apache.org>
Subject [jira] [Commented] (CURATOR-58) ACLs support for ZKPaths.mkdirs()
Date Fri, 08 Nov 2013 18:14:20 GMT

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

Robert Kanter commented on CURATOR-58:
--------------------------------------

Thanks for taking care of this.  

Btw, not a big deal, but the TestLockACLs test that I posted in the email thread and was added
to the tests has a comment that reads:
{code:java}
// Now try creating a lock and we'll see that it incorrectly has the default world ACLs
{code}
is no longer true because you fixed the problem.

> ACLs support for ZKPaths.mkdirs()
> ---------------------------------
>
>                 Key: CURATOR-58
>                 URL: https://issues.apache.org/jira/browse/CURATOR-58
>             Project: Apache Curator
>          Issue Type: Bug
>          Components: Client, Framework
>    Affects Versions: 2.0.1-incubating
>         Environment: Ubuntu 10.10
>            Reporter: Benjamin Jaton
>            Priority: Minor
>             Fix For: 2.3.1
>
>
> ZKPaths always creates parent node using the OPEN_ACL_UNSAFE ACL:
> line 168:
> zookeeper.create(subPath, new byte[0], ZooDefs.Ids.OPEN_ACL_UNSAFE, CreateMode.PERSISTENT);
> As a result:
> - EnsurePath.ensure(..)
> - CreateBuilderImpl.backgroundCreateParentsThenNode
> - CreateBuilderImpl.pathInForeground
> will never use the ACLProvider and will always create unprotected nodes.
> The workaround is not to use those methods and create the parents manually, node by node.



--
This message was sent by Atlassian JIRA
(v6.1#6144)

Mime
View raw message