Details

    • Type: Bug Bug
    • Status: Open
    • Priority: Major Major
    • Resolution: Unresolved
    • Affects Version/s: 3.3.1, 3.3.2, 3.3.3
    • Fix Version/s: 3.5.2, 3.6.0
    • Component/s: java client
    • Labels:
      None
    • Tags:
      acl-check

      Description

      I watched the source of the zookeeper class and I missed an acl check in the asynchronous version of the create operation. Is there any reason, that in the asynch version is no
      check whether the acl is valid, or did someone forget to implement it. It's interesting because we worked on a refactoring of the zookeeper client and don't want to implement a bug.

      The following code is missing:
      if (acl != null && acl.size() == 0)

      { throw new KeeperException.InvalidACLException(); }

        Activity

        Michi Mutsuzaki made changes -
        Fix Version/s 3.5.2 [ 12331981 ]
        Fix Version/s 3.6.0 [ 12326518 ]
        Fix Version/s 3.5.1 [ 12326786 ]
        Patrick Hunt made changes -
        Fix Version/s 3.5.1 [ 12326786 ]
        Fix Version/s 3.5.0 [ 12316644 ]
        Laxman made changes -
        Assignee Laxman [ lakshman ]
        Mahadev konar made changes -
        Fix Version/s 3.5.0 [ 12316644 ]
        Fix Version/s 3.4.0 [ 12314469 ]
        Mahadev konar made changes -
        Status Patch Available [ 10002 ] Open [ 1 ]
        Laxman made changes -
        Status Open [ 1 ] Patch Available [ 10002 ]
        Affects Version/s 3.3.3 [ 12315482 ]
        Affects Version/s 3.3.2 [ 12315108 ]
        Fix Version/s 3.4.0 [ 12314469 ]
        Tags ack-check acl-check
        Laxman made changes -
        Attachment ZOOKEEPER-847.patch [ 12492215 ]
        Laxman made changes -
        Assignee Thomas Koch [ thkoch ] Laxman [ lakshman ]
        Thomas Koch made changes -
        Assignee Thomas Koch [ thkoch ]
        Thomas Koch made changes -
        Field Original Value New Value
        Summary Missing ack-Check in Zookeeper create Missing acl check in zookeeper create
        Description I watched the source of zookeeper class and i missed an ack-check in the asynchronous version of the create operation. Is there any reason, that in the asynch version is no
        check whether the ack is valid, or did someone forget to implement it. It's interesting because we worked on a refactoring of the zookeeper client and don't want to implement a bug.
        I watched the source of the zookeeper class and I missed an acl check in the asynchronous version of the create operation. Is there any reason, that in the asynch version is no
        check whether the acl is valid, or did someone forget to implement it. It's interesting because we worked on a refactoring of the zookeeper client and don't want to implement a bug.

        The following code is missing:
                if (acl != null && acl.size() == 0) {
                    throw new KeeperException.InvalidACLException();
                }
        Patrick Datko created issue -

          People

          • Assignee:
            Unassigned
            Reporter:
            Patrick Datko
          • Votes:
            0 Vote for this issue
            Watchers:
            3 Start watching this issue

            Dates

            • Created:
              Updated:

              Development