zeroc-ice / ice

All-in-one solution for creating networked applications with RPC, pub/sub, server deployment, and more.
https://zeroc.com
GNU General Public License v2.0
2.04k stars 592 forks source link

Review peer certificate tests #2396

Open pepone opened 3 months ago

pepone commented 3 months ago

It's not clear why CI didn't catch this bug.

Yes, this was bogus we want the leaf certificate with index 0. The previous code overrides _peerCertificate for each cert in the chain which doesn't make sense.

_Originally posted by @pepone in https://github.com/zeroc-ice/ice/pull/2388#discussion_r1663192305_