zookeeper-dev mailing list archives

Site index · List index
Message view « Date » · « Thread »
Top « Date » · « Thread »
From hanm <...@git.apache.org>
Subject [GitHub] zookeeper pull request #182: ZOOKEEPER-2709 Clarify documentation around the...
Date Tue, 07 Mar 2017 06:13:22 GMT
Github user hanm commented on a diff in the pull request:

    https://github.com/apache/zookeeper/pull/182#discussion_r104595517
  
    --- Diff: src/docs/src/documentation/content/xdocs/zookeeperProgrammers.xml ---
    @@ -899,9 +899,12 @@
             single id, <emphasis>anyone</emphasis>, that represents
             anyone.</para></listitem>
     
    -        <listitem><para><emphasis role="bold">auth</emphasis>
doesn't
    -        use any id, represents any authenticated
    -        user.</para></listitem>
    +        <listitem><para><emphasis role="bold">auth</emphasis>
is a convenience
    +        scheme which defaults to the currently-authenticated user and scheme.
    +        Any ID which is provided using this scheme is ignored by ZooKeeper.
    --- End diff --
    
    Thanks for the patch, it is a good improvement on doc.
    
    I think the ID here refers to the id of the scheme:id pair of the ID object in the ACL,
correct? Mentioning this because the auth scheme is also referenced in command line where
people can do 'setAcl /node auth:username:password:crdwa' in which case the username (sometimes
overloaded as id) is required. 



---
If your project is set up for it, you can reply to this email and have your
reply appear on GitHub as well. If your project does not have this feature
enabled and wishes so, or if the feature is enabled but not working, please
contact infrastructure at infrastructure@apache.org or file a JIRA ticket
with INFRA.
---

Mime
View raw message