Skip to content

Fixes #4683 - uses X/Y instead of Gx/Gy for thumbprint of ecdsa keys#4684

Merged
wy65701436 merged 1 commit into
distribution:mainfrom
gpgenaiz:4683-invalid-thumbprint-of-ecdsa-keys
Aug 12, 2025
Merged

Fixes #4683 - uses X/Y instead of Gx/Gy for thumbprint of ecdsa keys#4684
wy65701436 merged 1 commit into
distribution:mainfrom
gpgenaiz:4683-invalid-thumbprint-of-ecdsa-keys

Conversation

@gpgenaiz

@gpgenaiz gpgenaiz commented Aug 5, 2025

Copy link
Copy Markdown
Contributor

Simply fixes registry/auth/token/util handling of ecdsa key prints

@milosgajdos milosgajdos requested a review from thaJeztah August 5, 2025 21:01

@milosgajdos milosgajdos left a comment

Copy link
Copy Markdown
Member

Choose a reason for hiding this comment

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

PTAL @thaJeztah

Comment thread registry/auth/token/util.go Outdated
@gpgenaiz gpgenaiz force-pushed the 4683-invalid-thumbprint-of-ecdsa-keys branch from dd1fbac to 0fb75cd Compare August 11, 2025 13:56
@milosgajdos milosgajdos requested a review from wy65701436 August 11, 2025 14:20
… ecdsa keys

Signed-off-by: Guillaume Pelletier <guillaume.pelletier@genaiz.com>
@gpgenaiz gpgenaiz force-pushed the 4683-invalid-thumbprint-of-ecdsa-keys branch from 0fb75cd to 6e59b82 Compare August 11, 2025 14:30
@milosgajdos

Copy link
Copy Markdown
Member

PTAL again @wy65701436

@milosgajdos milosgajdos requested a review from joaodrp August 11, 2025 22:45

@wy65701436 wy65701436 left a comment

Copy link
Copy Markdown
Collaborator

Choose a reason for hiding this comment

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

lgtm

@wy65701436 wy65701436 merged commit 8c60ad7 into distribution:main Aug 12, 2025
21 checks passed
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Projects

None yet

Development

Successfully merging this pull request may close these issues.

4 participants