Gh-2997: Prevented a NullPointerException being thrown.

[Banji0x]

This commit refactors the code to address the scenario where the zookeepers property is null. The modified code now checks if the zookeepers property is null, and if so, it sets the ZooKeeperPort to the default value. This change ensures that a NullPointerException is avoided when accessing the zookeepers property and provides a more robust behavior in the case of a missing or null value.

[CLAassistant]

All committers have signed the CLA.

[Banji0x]

@GCHQDeveloper314
It seems like some tests failed, how do i resolve that ?

[GCHQDeveloper314]

I believe they were cancelled because of force pushing to the branch. Please avoid force pushing, as we squash all commits on merge there's no need to force push. It can cause issues with the CI sometimes.

There's a need to update the copyright header on the file that was changed to include the current year.

[Banji0x]

@GCHQDeveloper314
Understood.
Copyright has been updated.
Kindly review changes.

[codecov]

Codecov Report

Merging #2998 (8971ce5) into develop (62d8331) will decrease coverage by 0.02%.
The diff coverage is n/a.

@@              Coverage Diff              @@
##             develop    #2998      +/-   ##
=============================================
- Coverage      65.17%   65.16%   -0.02%     
+ Complexity      2633     2631       -2     
=============================================
  Files            901      901              
  Lines          28797    28797              
  Branches        3238     3238              
=============================================
- Hits           18769    18765       -4     
- Misses          8572     8574       +2     
- Partials        1456     1458       +2     

see 2 files with indirect coverage changes

[Banji0x]

@GCHQDeveloper314 copyright has been updated.