search

nats-io / nats-server

High-Performance server for NATS.io, the cloud and edge native messaging system.
https://nats.io
Apache License 2.0
16k stars 1.41k forks source link

Fix matching by `thumbprint` in the Windows certificate store #6047

Closed neilalexander closed 3 weeks ago

neilalexander commented 3 weeks ago

Fixes a bug in #6042 where the thumbprint would never match, as we were passing in a hex-decoded form instead of the actual string value.

Signed-off-by: Neil Twigg neil@nats.io

neilalexander commented 3 weeks ago

You're right, there should have been. I had apparently made the change to the unit test in my Windows VM and not copied it back to the host Mac to commit.

The existing test case has been updated with the thumbprint now.