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

imghelper: disable pkix in nss #335

Closed
wants to merge 1 commit into from

Conversation

cbgbt
Copy link
Contributor

@cbgbt cbgbt commented Jul 25, 2024

Description of changes:
During image creation, bottlerocket uses the pesigcheck utility to verify secureboot signatures on all signed artifacts. The signature validation is implemented via libnss.

The next Bottlerocket SDK release will move to NSS 3.101.

In NSS 3.101, lib::pkix was enabled as the default X.509 validator. This causes pesigcheck to fail with "Peer's Certificate issuer is not recognized," despite the CA issuer being provided to pesigcheck.

This change sets NSS_DISABLE_PKIX_VERIFY=1, which reverts to the previous default verifier.

I've opened #334 to root-cause the changes in pkix which cause the issue. We will revert to the previous validator while working to become compatible with pkix.

Testing done:

  • Bottlerocket builds succeed using SDK 0.42.0 and test artifacts for 0.43.0.

Terms of contribution:

By submitting this pull request, I agree that this contribution is dual-licensed under the terms of both the Apache License, version 2.0, and the MIT license.

In NSS 3.101, lib::pkix was enabled as the default X.509 validator. This
causes pesigcheck to fail with "Peer's Certificate issuer is not
recognized," despite the CA issuer being provided to pesigcheck.

We will revert to the previous validator while working to become
compatible with pkix.
@cbgbt cbgbt closed this Jul 26, 2024
@cbgbt
Copy link
Contributor Author

cbgbt commented Jul 26, 2024

We've opted to instead include this fix in the SDK itself in bottlerocket-os/bottlerocket-sdk#194

Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
None yet
Projects
None yet
Development

Successfully merging this pull request may close these issues.

3 participants