You are viewing a plain text version of this content. The canonical link for it is here.
Posted to dev@zookeeper.apache.org by ivmaykov <gi...@git.apache.org> on 2018/11/20 23:25:40 UTC

[GitHub] zookeeper pull request #709: ZOOKEEPER-3194: Quorum TLS - fix copy/paste bug...

GitHub user ivmaykov opened a pull request:

    https://github.com/apache/zookeeper/pull/709

    ZOOKEEPER-3194: Quorum TLS - fix copy/paste bug in ZKTrustManager

    Fix an obvious copy/paste bug. Tested by making sure ZKTrustManagerTest still passes.

You can merge this pull request into a Git repository by running:

    $ git pull https://github.com/ivmaykov/zookeeper ZOOKEEPER-3194

Alternatively you can review and apply these changes as the patch at:

    https://github.com/apache/zookeeper/pull/709.patch

To close this pull request, make a commit to your master/trunk branch
with (at least) the following in the commit message:

    This closes #709
    
----
commit ad6502ddc47236c4db359e1d1fa5af3a48e2684c
Author: Ilya Maykov <il...@...>
Date:   2018-11-20T23:23:28Z

    ZOOKEEPER-3194: Quorum TLS - fix copy/paste bug in ZKTrustManager

----


---

[GitHub] zookeeper pull request #709: ZOOKEEPER-3194: Quorum TLS - fix copy/paste bug...

Posted by ivmaykov <gi...@git.apache.org>.
GitHub user ivmaykov reopened a pull request:

    https://github.com/apache/zookeeper/pull/709

    ZOOKEEPER-3194: Quorum TLS - fix copy/paste bug in ZKTrustManager

    Fix an obvious copy/paste bug. Tested by making sure ZKTrustManagerTest still passes.

You can merge this pull request into a Git repository by running:

    $ git pull https://github.com/ivmaykov/zookeeper ZOOKEEPER-3194

Alternatively you can review and apply these changes as the patch at:

    https://github.com/apache/zookeeper/pull/709.patch

To close this pull request, make a commit to your master/trunk branch
with (at least) the following in the commit message:

    This closes #709
    
----
commit ad6502ddc47236c4db359e1d1fa5af3a48e2684c
Author: Ilya Maykov <il...@...>
Date:   2018-11-20T23:23:28Z

    ZOOKEEPER-3194: Quorum TLS - fix copy/paste bug in ZKTrustManager

----


---

[GitHub] zookeeper pull request #709: ZOOKEEPER-3194: Quorum TLS - fix copy/paste bug...

Posted by ivmaykov <gi...@git.apache.org>.
Github user ivmaykov closed the pull request at:

    https://github.com/apache/zookeeper/pull/709


---

[GitHub] zookeeper pull request #709: ZOOKEEPER-3194: Quorum TLS - fix copy/paste bug...

Posted by ivmaykov <gi...@git.apache.org>.
GitHub user ivmaykov reopened a pull request:

    https://github.com/apache/zookeeper/pull/709

    ZOOKEEPER-3194: Quorum TLS - fix copy/paste bug in ZKTrustManager

    Fix an obvious copy/paste bug. Tested by making sure ZKTrustManagerTest still passes.

You can merge this pull request into a Git repository by running:

    $ git pull https://github.com/ivmaykov/zookeeper ZOOKEEPER-3194

Alternatively you can review and apply these changes as the patch at:

    https://github.com/apache/zookeeper/pull/709.patch

To close this pull request, make a commit to your master/trunk branch
with (at least) the following in the commit message:

    This closes #709
    
----
commit ad6502ddc47236c4db359e1d1fa5af3a48e2684c
Author: Ilya Maykov <il...@...>
Date:   2018-11-20T23:23:28Z

    ZOOKEEPER-3194: Quorum TLS - fix copy/paste bug in ZKTrustManager

----


---

[GitHub] zookeeper issue #709: ZOOKEEPER-3194: Quorum TLS - fix copy/paste bug in ZKT...

Posted by ivmaykov <gi...@git.apache.org>.
Github user ivmaykov commented on the issue:

    https://github.com/apache/zookeeper/pull/709
  
    @anmolnar could you merge this into master and branch-3.5?


---

[GitHub] zookeeper pull request #709: ZOOKEEPER-3194: Quorum TLS - fix copy/paste bug...

Posted by ivmaykov <gi...@git.apache.org>.
Github user ivmaykov closed the pull request at:

    https://github.com/apache/zookeeper/pull/709


---

[GitHub] zookeeper issue #709: ZOOKEEPER-3194: Quorum TLS - fix copy/paste bug in ZKT...

Posted by anmolnar <gi...@git.apache.org>.
Github user anmolnar commented on the issue:

    https://github.com/apache/zookeeper/pull/709
  
    Committed. Thanks @ivmaykov !


---

[GitHub] zookeeper pull request #709: ZOOKEEPER-3194: Quorum TLS - fix copy/paste bug...

Posted by asfgit <gi...@git.apache.org>.
Github user asfgit closed the pull request at:

    https://github.com/apache/zookeeper/pull/709


---