Skip to content

Commit 3017dfc

Browse files
committed
Fixes #1201 - Raise an explicit error when only env var AWS_ACCESS_KEY is set
1 parent 0e9044b commit 3017dfc

File tree

1 file changed

+5
-0
lines changed

1 file changed

+5
-0
lines changed

S3/Config.py

Lines changed: 5 additions & 0 deletions
Original file line numberDiff line numberDiff line change
@@ -273,6 +273,11 @@ def __init__(self, configfile = None, access_key=None, secret_key=None, access_t
273273
env_secret_key = os.getenv('AWS_SECRET_KEY') or os.getenv('AWS_SECRET_ACCESS_KEY')
274274
env_access_token = os.getenv('AWS_SESSION_TOKEN') or os.getenv('AWS_SECURITY_TOKEN')
275275
if env_access_key:
276+
if not env_secret_key:
277+
raise ValueError(
278+
"AWS_ACCESS_KEY environment variable is used but"
279+
" AWS_SECRET_KEY variable is missing"
280+
)
276281
# py3 getenv returns unicode and py2 returns bytes.
277282
self.access_key = base_unicodise(env_access_key)
278283
self.secret_key = base_unicodise(env_secret_key)

0 commit comments

Comments
 (0)