[15012]: Bug fix, small refactor and tests in TlsConfiguration

[antonrv]

• Bug fix according to reported issue: TLS config for both server and client does not work #243

• Small refactor to simplify TlsConfiguration architecture (bloated class hierarchy, unneeded polymorphism)

• New tests for previously untested scenarios

[codecov]

Codecov Report

Merging #244 (55e2f37) into main (ce4904f) will increase coverage by 0.42%.
The diff coverage is 66.21%.

@@            Coverage Diff             @@
##             main     #244      +/-   ##
==========================================
+ Coverage   58.88%   59.31%   +0.42%     
==========================================
  Files          93       91       -2     
  Lines        3223     3200      -23     
  Branches     1181     1188       +7     
==========================================
  Hits         1898     1898              
+ Misses        599      574      -25     
- Partials      726      728       +2     

Impacted Files	Coverage Δ
ddsrouter_yaml/src/cpp/YamlReader.cpp	42.28% <16.66%> (+0.56%)	⬆️
...entations/rtps/impl/DiscoveryServerParticipant.ipp	47.88% <47.05%> (-0.03%)	⬇️
...cipant/DiscoveryServerParticipantConfiguration.cpp	48.93% <66.66%> (+6.34%)	⬆️
...re/src/cpp/types/security/tls/TlsConfiguration.cpp	80.00% <78.57%> (+33.33%)	⬆️
...outer_core/types/security/tls/TlsConfiguration.hpp	100.00% <100.00%> (ø)
ddsrouter_core/src/cpp/efficiency/PayloadPool.cpp	76.47% <0.00%> (-1.97%)	⬇️
ddsrouter_core/src/cpp/communication/Track.cpp	64.86% <0.00%> (+2.70%)	⬆️

---

Continue to review full report at Codecov.

Legend - Click here to learn more
Δ = absolute <relative> (impact), ø = not affected, ? = missing data
Powered by Codecov. Last update ce4904f...55e2f37. Read the comment docs.

[jparisu]

Cool and elegant Implementation 👍
Just minor suggestions.

[jparisu]

Everything LGTM,
Windows fails linking new methods nevertheless.

[rsanchez15]

Do not forget to add this bugfix to the release notes.

[jparisu]

LGTM with green CI