[GitHub] incubator-pulsar pull request #530: Support wildcard in permitted role name

2017-06-27 Thread yush1ga
Github user yush1ga commented on a diff in the pull request: https://github.com/apache/incubator-pulsar/pull/530#discussion_r124403361 --- Diff: conf/broker.conf --- @@ -140,6 +140,10 @@ authenticationProviders= # Enforce authorization authorizationEnabled=false

[GitHub] incubator-pulsar pull request #530: Support wildcard in permitted role name

2017-06-27 Thread yush1ga
Github user yush1ga commented on a diff in the pull request: https://github.com/apache/incubator-pulsar/pull/530#discussion_r124403361 --- Diff: conf/broker.conf --- @@ -140,6 +140,10 @@ authenticationProviders= # Enforce authorization authorizationEnabled=false

[GitHub] incubator-pulsar pull request #530: Support wildcard in permitted role name

2017-06-26 Thread yush1ga
Github user yush1ga commented on a diff in the pull request: https://github.com/apache/incubator-pulsar/pull/530#discussion_r124128740 --- Diff: pulsar-broker-common/src/main/java/org/apache/pulsar/broker/authorization/AuthorizationManager.java --- @@ -177,6 +190,29 @@ private

[GitHub] incubator-pulsar pull request #530: Support wildcard in permitted role name

2017-06-26 Thread yush1ga
Github user yush1ga commented on a diff in the pull request: https://github.com/apache/incubator-pulsar/pull/530#discussion_r124128740 --- Diff: pulsar-broker-common/src/main/java/org/apache/pulsar/broker/authorization/AuthorizationManager.java --- @@ -177,6 +190,29 @@ private

[GitHub] incubator-pulsar pull request #530: Support wildcard in permitted role name

2017-06-26 Thread yush1ga
Github user yush1ga commented on a diff in the pull request: https://github.com/apache/incubator-pulsar/pull/530#discussion_r124128272 --- Diff: conf/broker.conf --- @@ -140,6 +140,10 @@ authenticationProviders= # Enforce authorization authorizationEnabled=false

[GitHub] incubator-pulsar pull request #530: Support wildcard in permitted role name

2017-06-26 Thread yush1ga
Github user yush1ga commented on a diff in the pull request: https://github.com/apache/incubator-pulsar/pull/530#discussion_r124128308 --- Diff: pulsar-broker-common/src/main/java/org/apache/pulsar/broker/authorization/AuthorizationManager.java --- @@ -177,6 +190,29 @@ private

[GitHub] incubator-pulsar pull request #530: Support wildcard in permitted role name

2017-06-26 Thread yush1ga
Github user yush1ga commented on a diff in the pull request: https://github.com/apache/incubator-pulsar/pull/530#discussion_r124128272 --- Diff: conf/broker.conf --- @@ -140,6 +140,10 @@ authenticationProviders= # Enforce authorization authorizationEnabled=false

[GitHub] incubator-pulsar pull request #530: Support wildcard in permitted role name

2017-06-26 Thread yush1ga
Github user yush1ga commented on a diff in the pull request: https://github.com/apache/incubator-pulsar/pull/530#discussion_r124128308 --- Diff: pulsar-broker-common/src/main/java/org/apache/pulsar/broker/authorization/AuthorizationManager.java --- @@ -177,6 +190,29 @@ private

[GitHub] incubator-pulsar pull request #530: Support wildcard in permitted role name

2017-06-26 Thread yush1ga
Github user yush1ga commented on a diff in the pull request: https://github.com/apache/incubator-pulsar/pull/530#discussion_r124128153 --- Diff: pulsar-broker/src/test/java/org/apache/pulsar/broker/auth/AuthorizationTest.java --- @@ -94,6 +96,46 @@ void simple() throws Exception

[GitHub] incubator-pulsar pull request #530: Support wildcard in permitted role name

2017-06-26 Thread yush1ga
Github user yush1ga commented on a diff in the pull request: https://github.com/apache/incubator-pulsar/pull/530#discussion_r124128153 --- Diff: pulsar-broker/src/test/java/org/apache/pulsar/broker/auth/AuthorizationTest.java --- @@ -94,6 +96,46 @@ void simple() throws Exception

[GitHub] incubator-pulsar pull request #530: Support wildcard in permitted role name

2017-06-26 Thread yush1ga
GitHub user yush1ga opened a pull request: https://github.com/apache/incubator-pulsar/pull/530 Support wildcard in permitted role name This PR is resolving conflicts of #494 as I commented when closing it. ### Motivation Please refer to #469

[GitHub] incubator-pulsar pull request #494: Support wildcard in permitted role name

2017-06-26 Thread yush1ga
Github user yush1ga closed the pull request at: https://github.com/apache/incubator-pulsar/pull/494 --- 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

[GitHub] incubator-pulsar issue #494: Support wildcard in permitted role name

2017-06-26 Thread yush1ga
Github user yush1ga commented on the issue: https://github.com/apache/incubator-pulsar/pull/494 Although I resolved conflicts, it's not reflected here. I will close this and re-send PR. --- If your project is set up for it, you can reply to this email and have your reply a

[GitHub] incubator-pulsar issue #494: Support wildcard in permitted role name

2017-06-26 Thread yush1ga
Github user yush1ga commented on the issue: https://github.com/apache/incubator-pulsar/pull/494 Although I resolved conflicts, it's not reflected here. I will close this and re-send PR. --- If your project is set up for it, you can reply to this email and have your reply a

[GitHub] incubator-pulsar pull request #494: Support wildcard in permitted role name

2017-06-26 Thread yush1ga
Github user yush1ga closed the pull request at: https://github.com/apache/incubator-pulsar/pull/494 --- 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

[GitHub] incubator-pulsar issue #504: TLS test for C++ client is intermittently faili...

2017-06-21 Thread yush1ga
Github user yush1ga commented on the issue: https://github.com/apache/incubator-pulsar/issues/504 I couldn't reproduce it in my local environment (Mac) ... Do you have an idea to reproduce ? --- If your project is set up for it, you can reply to this email and have your