Skip to content

Add --passphrase-fd option.#276

Closed
sz-coder wants to merge 4 commits into
FiloSottile:masterfrom
sz-coder:master
Closed

Add --passphrase-fd option.#276
sz-coder wants to merge 4 commits into
FiloSottile:masterfrom
sz-coder:master

Conversation

@sz-coder
Copy link
Copy Markdown

@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
Copy Markdown
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
Copy Markdown
Owner

#275 (comment)

@clach04
Copy link
Copy Markdown

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