#2091 Remove redirection of stderr when calling credential_process

Open itsdalmo
Showing 1 of 2 files from the diff.

@@ -964,13 +964,11 @@
Loading
964 964
        # We're not using shell=True, so we need to pass the
965 965
        # command and all arguments as a list.
966 966
        process_list = compat_shell_split(credential_process)
967 -
        p = self._popen(process_list,
968 -
                        stdout=subprocess.PIPE,
969 -
                        stderr=subprocess.PIPE)
970 -
        stdout, stderr = p.communicate()
967 +
        p = self._popen(process_list, stdout=subprocess.PIPE)
968 +
        stdout, _ = p.communicate()
971 969
        if p.returncode != 0:
972 970
            raise CredentialRetrievalError(
973 -
                provider=self.METHOD, error_msg=stderr.decode('utf-8'))
971 +
                provider=self.METHOD, error_msg=("non-zero exit code '%s'" % p.returncode))
974 972
        parsed = botocore.compat.json.loads(stdout.decode('utf-8'))
975 973
        version = parsed.get('Version', '<Version key not provided>')
976 974
        if version != 1:

Learn more Showing 1 files with coverage changes found.

Changes in botocore/credentials.py
-3
+3
Loading file...
Files Coverage
botocore 0.03% 92.75%
Project Totals (58 files) 92.75%
Loading