removed support for stdin credentials

awsume/__data__.py

@@ -1,4 +1,4 @@
-version = '4.1.7'
+version = '4.1.8'
 
 name = 'awsume'
 author = 'Trek10, Inc'

awsume/awsumepy/app.py

@@ -179,11 +179,10 @@ def get_credentials(self, args: argparse.Namespace, profiles: dict) -> dict:
             profiles=profiles,
         )
         try:
-            if not args.auto_refresh and (args.json or not sys.stdin.isatty()): # piping/sending credentials to awsume directly
-                logger.debug('Pulling credentials from {}'.format('json parameter' if args.json else 'stdin'))
-                args.target_profile_name = 'json' if args.json else 'stdin'
-                json_input = args.json if args.json else sys.stdin.read()
-                credentials = json.loads(json_input)
+            if not args.auto_refresh and args.json: # sending credentials to awsume directly
+                logger.debug('Pulling credentials from json parameter')
+                args.target_profile_name = 'json'
+                credentials = json.loads(args.json)
                 if 'Credentials' in credentials:
                     credentials = credentials['Credentials']
                 credentials = [credentials]

awsume/awsumepy/default_plugins.py

@@ -187,8 +187,11 @@ def add_arguments(config: dict, parser: argparse.ArgumentParser):
 def post_add_arguments(config: dict, arguments: argparse.Namespace, parser: argparse.ArgumentParser):
     logger.debug('Post add arguments')
     logger.debug(json.dumps(vars(arguments)))
-    if arguments.role_arn and arguments.auto_refresh:
-        raise exceptions.ValidationException('Cannot use autoawsume with given role_arn')
+    if arguments.auto_refresh:
+        if arguments.role_arn:
+            raise exceptions.ValidationException('Cannot use autoawsume with a given role_arn')
+        if arguments.json:
+            raise exceptions.ValidationException('Cannot use autoawsume with json')
     if arguments.version:
         logger.debug('Logging version')
         safe_print(__data__.version)
@@ -244,7 +247,6 @@ def post_add_arguments(config: dict, arguments: argparse.Namespace, parser: argp
             parser.error('--principal-arn account id must be valid numeric account id of length 12')
         arguments.principal_arn = 'arn:{}:iam::{}:role/{}'.format(partition, account_id, provider_name)
 
-
     if not arguments.profile_name:
         if arguments.role_arn:
             logger.debug('Role arn passed, target profile name will be role_arn')

docs/changelog.md

@@ -1,5 +1,9 @@
 # Awsume Changelog
 
+## [4.1.8] - WIP - Bug Fix
+
+- Removed support for piping credentials to stdin
+
 ## [4.1.7] - 2019-10-02 - Autoawsume Bug Fix
 
 - Fixed autoawsume bug with datetime comparisons