Skip to content
New issue

Have a question about this project? Sign up for a free GitHub account to open an issue and contact its maintainers and the community.

By clicking “Sign up for GitHub”, you agree to our terms of service and privacy statement. We’ll occasionally send you account related emails.

Already on GitHub? Sign in to your account

UNTESTED DO NOT MERGE: Update for golang-fips API changes #2305

Closed
wants to merge 1 commit into from

Conversation

mtrmac
Copy link
Contributor

@mtrmac mtrmac commented Apr 18, 2024

Trying to see what a fix for the Packit-reported failures (as in #2297 (comment) ) would look like.

@lsm5 does this build on all distributions we care about for the main branch of our products?


This passes libtrust tests on macOS, I need to also test this with the OpenSSl backend.

@mtrmac mtrmac mentioned this pull request Apr 18, 2024
Copy link

Ephemeral COPR build failed. @containers/packit-build please check.

@mtrmac
Copy link
Contributor Author

mtrmac commented Apr 18, 2024

From a quick analysis, the ecdsa.HashSign function referred to in #2297 (comment) seems not to be public.

For reference, a (private) conversation motivating FIPS variant: https://bugzilla.redhat.com/show_bug.cgi?id=1740920 .

@lsm5
Copy link
Member

lsm5 commented Apr 22, 2024

sorry about the delay. my github notifications were messed up. I'll look into this today.

This passes libtrust tests on macOS, need to also test this with
the OpenSSl backend.

Signed-off-by: Miloslav Trmač <[email protected]>
@mtrmac
Copy link
Contributor Author

mtrmac commented May 9, 2024

Follow containers/libtrust#8 for current progress on these build failures.

@mtrmac
Copy link
Contributor Author

mtrmac commented Jun 6, 2024

This was fixed in golang-fips, culminating in containers/libtrust#8 .

@mtrmac mtrmac closed this Jun 6, 2024
@mtrmac mtrmac deleted the libtrust-update branch June 6, 2024 14:19
@stale-locking-app stale-locking-app bot locked as resolved and limited conversation to collaborators Sep 5, 2024
Sign up for free to subscribe to this conversation on GitHub. Already have an account? Sign in.
Projects
None yet
Development

Successfully merging this pull request may close these issues.

2 participants