[
https://issues.apache.org/jira/browse/ZOOKEEPER-2792?page=com.atlassian.jira.plugin.system.issuetabpanels:comment-tabpanel&focusedCommentId=16097106#comment-16097106
]
Rakesh R commented on ZOOKEEPER-2792:
-------------------------------------
bq. provide an upgrade path from 3.4 to 3.5, so what it aimed to test (the
upgrade logic) is only applicable to branch-3.4
IIUC, server will only permit a quorum connection from another server, if the
other server has a lower sid. Please refer
[QuorumCnxManager.java#L354|https://github.com/apache/zookeeper/blob/branch-3.5/src/java/main/org/apache/zookeeper/server/quorum/QuorumCnxManager.java#L354]
logic. {{QuorumCnxManagerTest}} test class contains many test cases related to
this. Also, we have other set of test cases verifying the auth connection
behaviors based on the {{serverRequireSasl}} and {{learnerRequireSasl}} flags.
I think, {{QuorumCnxManagerTest}} cases would be a good addition by covering
these cases in {{3.5}} as well. Since the current merge patch is large, I agree
to push this via separate su-task.
> [QP MutualAuth]: Port ZOOKEEPER-1045 implementation from branch-3.4 to
> branch-3.5
> ---------------------------------------------------------------------------------
>
> Key: ZOOKEEPER-2792
> URL: https://issues.apache.org/jira/browse/ZOOKEEPER-2792
> Project: ZooKeeper
> Issue Type: Sub-task
> Components: quorum, security
> Reporter: Rakesh R
> Assignee: Michael Han
> Fix For: 3.5.4, 3.6.0
>
> Attachments: ZOOKEEPER-2792.patch
>
>
> This jira is to merge the basic working patch covering the authentication and
> authorization of static(zoo.cfg) ZooKeeper servers from {{branch-3.4}} code
> base.
--
This message was sent by Atlassian JIRA
(v6.4.14#64029)
