From b33fbd6d8ced4f8ae19839e9a9e7988906a70f36 Mon Sep 17 00:00:00 2001 From: Thomas Waldmann Date: Tue, 20 Mar 2018 21:21:23 +0100 Subject: [PATCH] improve getpass user experience, see #3689 if interactive passphrase query fails and the env vars are not set, show a clear error message about this. users often do 'BORG_PASSPHRASE=secret', forgetting the 'export'. or they use sudo (and not sudo -E). in both cases, the env vars won't be available for the borg process. --- src/borg/crypto/key.py | 18 +++++++++++++++++- 1 file changed, 17 insertions(+), 1 deletion(-) diff --git a/src/borg/crypto/key.py b/src/borg/crypto/key.py index ed1d44fe7..c9de066e2 100644 --- a/src/borg/crypto/key.py +++ b/src/borg/crypto/key.py @@ -32,6 +32,10 @@ from .low_level import AES, bytes_to_long, bytes_to_int, num_aes_blocks, hmac_sh PREFIX = b'\0' * 8 +class NoPassphraseFailure(Error): + """can not acquire a passphrase: {}""" + + class PassphraseWrong(Error): """passphrase supplied in BORG_PASSPHRASE or by BORG_PASSCOMMAND is incorrect.""" @@ -451,7 +455,19 @@ class Passphrase(str): @classmethod def getpass(cls, prompt): - return cls(getpass.getpass(prompt)) + try: + pw = getpass.getpass(prompt) + except EOFError: + if prompt: + print() # avoid err msg appearing right of prompt + msg = [] + for env_var in 'BORG_PASSPHRASE', 'BORG_PASSCOMMAND': + env_var_set = os.environ.get(env_var) is not None + msg.append('%s is %s.' % (env_var, 'set' if env_var_set else 'not set')) + msg.append('Interactive password query failed.') + raise NoPassphraseFailure(' '.join(msg)) from None + else: + return cls(pw) @classmethod def verification(cls, passphrase):