From 3017dfc797b541feaceb06ff2eba6937ce5c325b Mon Sep 17 00:00:00 2001 From: Florent Viard Date: Mon, 27 Sep 2021 02:19:32 +0200 Subject: [PATCH] Fixes #1201 - Raise an explicit error when only env var AWS_ACCESS_KEY is set --- S3/Config.py | 5 +++++ 1 file changed, 5 insertions(+) diff --git a/S3/Config.py b/S3/Config.py index 8bafce44..68c14174 100644 --- a/S3/Config.py +++ b/S3/Config.py @@ -273,6 +273,11 @@ def __init__(self, configfile = None, access_key=None, secret_key=None, access_t env_secret_key = os.getenv('AWS_SECRET_KEY') or os.getenv('AWS_SECRET_ACCESS_KEY') env_access_token = os.getenv('AWS_SESSION_TOKEN') or os.getenv('AWS_SECURITY_TOKEN') if env_access_key: + if not env_secret_key: + raise ValueError( + "AWS_ACCESS_KEY environment variable is used but" + " AWS_SECRET_KEY variable is missing" + ) # py3 getenv returns unicode and py2 returns bytes. self.access_key = base_unicodise(env_access_key) self.secret_key = base_unicodise(env_secret_key)