CI Results:
All Go tests succeeded! ✅

Build Results:
All builds succeeded! ✅

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

👍

Is there maybe a unit test we can augment to cover this case?

Unit test would be welcome, there's also a similar error within GetPrivateKeyTypeFromPublicKey.

Would you mind addressing that one as well as part of this? Also what are your thoughts on just adding both the pointer and non-pointer types to all the case statements?

Also what are your thoughts on just adding both the pointer and non-pointer types to all the case statements?

I don't have a problem with it, but I don't know how useful it would be. I don't see anywhere that we expect an ed25519 key to be a pointer (except the case here)

I don't have a problem with it, but I don't know how useful it would be. I don't see anywhere that we expect an ed25519 key to be a pointer (except the case here)

Fair, lets not bother for now.