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

Add --passphrase-fd option. #276

Closed
wants to merge 4 commits into from
Closed

Conversation

sz-coder
Copy link

@sz-coder sz-coder commented May 31, 2021

This is my first attempt to get age reading a passphrase from a file descriptor.

See discussion.

@sz-coder
Copy link
Author

sz-coder commented Jun 1, 2021

Of course there are still some caveats that need to be addressed:

  • New line character (0xA) should be trimmed when reading from a file descriptor. (I think?)
  • Emit a warning if the passphrase does not fit completely in the buffer (currently 1024 bytes).
  • Move logic out of passphrasePromptForEncryption function.
  • Compatibility with other operating systems such as windows (I don't know if windows has the concepts of file descriptors).

@sz-coder sz-coder changed the title Added --passphrase-fd option for encrypting. Add --passphrase-fd option. Jun 1, 2021
@FiloSottile
Copy link
Owner

#275 (comment)

@clach04
Copy link

clach04 commented Nov 25, 2024

...

  • Compatibility with other operating systems such as windows (I don't know if windows has the concepts of file descriptors).

This would be a non-starter on Windows (but I can see the use for non-Windows).

Cross-referencing related/duplicated discussion/PR topics:

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