From a4f2e1cfdb16ddfff346c222ab8a8579f6d8aa76 Mon Sep 17 00:00:00 2001 From: Anders Ingemann Date: Tue, 10 Jul 2018 15:05:09 +0200 Subject: [PATCH] ec2: Fix error when credentials fetching fails A missing 'None' default would cover up the issue that the user has not supplied any credentials for ec2 --- bootstrapvz/providers/ec2/tasks/connection.py | 6 ++++-- 1 file changed, 4 insertions(+), 2 deletions(-) diff --git a/bootstrapvz/providers/ec2/tasks/connection.py b/bootstrapvz/providers/ec2/tasks/connection.py index 61e9e3b..dbf4973 100644 --- a/bootstrapvz/providers/ec2/tasks/connection.py +++ b/bootstrapvz/providers/ec2/tasks/connection.py @@ -55,8 +55,10 @@ class GetCredentials(Task): if provider_args.get('profile_name') not in Session().available_profiles: raise RuntimeError(( 'Profile specified was not found: {}'.format(provider_args.get('profile_name')))) - provider = Session(**provider_args).get_credentials().get_frozen_credentials() - if all(getattr(provider, provider_key(key)) is not None for key in keys): + provider = Session(**provider_args).get_credentials() + if provider is not None: + provider = provider.get_frozen_credentials() + if all(getattr(provider, provider_key(key), None) is not None for key in keys): for key in keys: creds[key] = getattr(provider, provider_key(key)) if hasattr(provider, 'token'):