common icon indicating copy to clipboard operation
common copied to clipboard

Published 20 hours ago • verified publisher icon prometheus

Fix bug for TLSVersion.MarshalYAML

Open StevenYCChou opened this issue 2 years ago • 2 comments

Signed-off-by: Yen-Cheng Chou [email protected]

StevenYCChou avatar Oct 14 '22 18:10 StevenYCChou

@roidelapluie PTAL.

StevenYCChou avatar Oct 14 '22 18:10 StevenYCChou

While there's a bug, I'm not sure that the fix is correct. I believe that MarshalJSON() needs to be changed to a value receiver function to work properly. I've experiment a bit with the Go playground: https://go.dev/play/p/P9nFVvIu-Mb

I agree

roidelapluie avatar Nov 03 '22 14:11 roidelapluie