Use secureClientPort instead of clientPort

[maltesander]

Currently for TLS and authentication we use the clientPort instead of secureClientPort due to a bug in ZooKeeper (https://issues.apache.org/jira/browse/ZOOKEEPER-4276).

Once this is fixed and we should switch from clientPort to secureClientPort and remove the portUnification property (allows plaintext and secured communication via the same port).

The necessary steps are documented in the code (see rust/crd/src/lib.rs)

Since this does not work in versions 3.8.0 or lower, it will be a little more work to differentiate between zookeeper versions with and without that bug.

Relates to PR #479.

Tasks

• fix(zookeeper): Apply patch ZOOKEEPER-4276 docker-images#1359
• Adapt to ZOOKEEPER-4276 #996

[sbernauer]

Good news, ZOOKEEPER-4276 was fixed and will be released in 3.10.0 🥳

[lfrancke]

As ZK 3.10.0 is not released yet (and we do not know when/if it will) the scope of this is to see if the patch applies on 3.9.x

Reading the comments I doubt this will be the case. Please spend at moste one day on trying to get the patch to apply and if that doesn't work raise it for discussion and a decision on how to continue.