Skip to content

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

Closed
mtrmac wants to merge 1 commit intocontainers:mainfrom
mtrmac:libtrust-update
Closed

UNTESTED DO NOT MERGE: Update for golang-fips API changes#2305
mtrmac wants to merge 1 commit intocontainers:mainfrom
mtrmac:libtrust-update

Conversation

@mtrmac
Copy link
Copy Markdown
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
@packit-as-a-service
Copy link
Copy Markdown

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

@mtrmac
Copy link
Copy Markdown
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
Copy Markdown
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č <mitr@redhat.com>
@mtrmac
Copy link
Copy Markdown
Contributor Author

mtrmac commented May 9, 2024

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

@mtrmac
Copy link
Copy Markdown
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