[GitHub] [skywalking] dmsolr commented on issue #3273: Support Zookeeper ACL

2019-08-18 Thread GitBox
dmsolr commented on issue #3273: Support Zookeeper ACL URL: https://github.com/apache/skywalking/pull/3273#issuecomment-522430444 We use Zookeeper ACL that has 2 steps, 1. set ACL for `zNode`. 2. authorization, like `login`. `IP` schema, we need not `authorization` step. But I

[GitHub] [skywalking] dmsolr commented on issue #3273: Support Zookeeper ACL

2019-08-18 Thread GitBox
dmsolr commented on issue #3273: Support Zookeeper ACL URL: https://github.com/apache/skywalking/pull/3273#issuecomment-522436577 I think whether we support `digest` only. Because `digest` is more widely used, and it is more clear. -

[GitHub] [skywalking] dmsolr commented on issue #3273: Support Zookeeper ACL

2019-08-18 Thread GitBox
dmsolr commented on issue #3273: Support Zookeeper ACL URL: https://github.com/apache/skywalking/pull/3273#issuecomment-522436839 How do you think? @wu-sheng @kezhenxu94 This is an automated message from the Apache Git Servic