Skip to content

verifier: humility isn't a tempfile#358

Merged
flihp merged 1 commit intomainfrom
ixi/error-messages
Mar 21, 2026
Merged

verifier: humility isn't a tempfile#358
flihp merged 1 commit intomainfrom
ixi/error-messages

Conversation

@iximeow
Copy link
Member

@iximeow iximeow commented Mar 21, 2026

@jordanhendricks just tripped over this as we snagging a cert from a racklette RoT. it turns out you run this from somewhere that can humility the system of interest, not on the system of interest, but getting an error about a TempFile(.. no such file or directory ..) was confusing!

I haven't actually run this (yet) but the types line up that failing to run the command would accidentally get reported as a tempfile thing unintentionally. wouldn't mind an actual check before merging (if Jordan has a sec?)

Copy link
Collaborator

@flihp flihp left a comment

Choose a reason for hiding this comment

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

This is great, thanks @iximeow. Looks like I missed this case when converting this crate from anyhow to the ctrl plane's / john's error handling guide lines. I'll press the merge button to keep things moving.

@flihp flihp merged commit 985ca75 into main Mar 21, 2026
16 checks passed
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.

2 participants