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

permslip signer clutters stdout #206

Open
flihp opened this issue Apr 19, 2024 · 0 comments
Open

permslip signer clutters stdout #206

flihp opened this issue Apr 19, 2024 · 0 comments

Comments

@flihp
Copy link
Collaborator

flihp commented Apr 19, 2024

This is cosmetic. The permslip signer dumps some stuff to stdout that we should ignore. Instead it ends up cluttering output from the manufacture command:

$ dice-mfg manufacture --require-release-policy false permslip
sending ping ... success
setting platform id to: "PDV2:PPP-PPPPPPP:RRR:0000000000B" ... success
getting CSR ... success
checking CSR ... success
Wrote to 0000000000B/ca.cert.pem
setting Intermediate cert ... success
Wrote to 0000000000B/PDV2:PPP-PPPPPPP:RRR:0000000000B.cert.pem
success
setting PlatformId cert ... success
sending Break ... success

The lines we should be ignoring are: Wrote to 0000000000B/ca.cert.pem and Wrote to 0000000000B/PDV2:PPP-PPPPPPP:RRR:0000000000B.cert.pem. Looks like there's an extra success message too?

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

No branches or pull requests

1 participant