fix to make action still run

Author: lehmanmj

README.md

@@ -112,7 +112,7 @@ See [action.yml](./action.yml) for more detail.
 | inline-session-policy     | You may further restrict the assumed role policy by defining an inline policy here.               |    No    |
 | managed-session-policies  | You may further restrict the assumed role policy by specifying a managed policy here.             |    No    |
 | output-credentials        | When set, outputs fetched credentials as action step output. (Outputs access-key-id, secret-access-key, session-token, and expiration). Defaults to false.                  |    No    |
-| output-env-credentials       | When set, exports fetched credentials as environment variables (AWS_REGION, AWS_DEFAULT_REGION, AWS_ACCESS_KEY_ID, AWS_SECRET_ACCESS_KEY, AWS_SESSION_TOKEN). Defaults to true. Set to false if you need to avoid setting/changing env variables. (You'd probably want to use output-credentials if you disable this). |    No    |
+| output-env-credentials       | When set, outputs fetched credentials as environment variables (AWS_REGION, AWS_DEFAULT_REGION, AWS_ACCESS_KEY_ID, AWS_SECRET_ACCESS_KEY, AWS_SESSION_TOKEN). Defaults to true. Set to false if you need to avoid setting/changing env variables. You'd probably want to use output-credentials if you disable this. (NOTE: Setting to false will prevent the aws-account-id from being exported as a step output). |    No    |
 | unset-current-credentials | When set, attempts to unset any existing credentials in your action runner.                       |    No    |
 | disable-retry             | Disabled retry/backoff logic for assume role calls. By default, retries are enabled.              |    No    |
 | retry-max-attempts        | Limits the number of retry attempts before giving up. Defaults to 12.                             |    No    |

dist/cleanup/index.js

@@ -63,7 +63,6 @@ export class CredentialsClient {
   }
 
   private async loadCredentials() {
-    info('trying to load credentials');
     const client = new STSClient({
       requestHandler: this.requestHandler ? this.requestHandler : undefined,
     });

dist/index.js

@@ -96,9 +96,7 @@ export function exportRegion(region: string, outputEnvCredentials?: boolean) {
 // Obtains account ID from STS Client and sets it as output
 export async function exportAccountId(credentialsClient: CredentialsClient, maskAccountId?: boolean) {
   const client = credentialsClient.stsClient;
-  core.info('trying to get account id');
   const identity = await client.send(new GetCallerIdentityCommand({}));
-  core.info('got account id');
   const accountId = identity.Account;
   if (!accountId) {
     throw new Error('Could not get Account ID from STS. Did you set credentials?');

src/CredentialsClient.ts

@@ -203,9 +203,9 @@ export async function run() {
       if (!process.env.GITHUB_ACTIONS || AccessKeyId) {
         await credentialsClient.validateCredentials(roleCredentials.Credentials?.AccessKeyId);
       }
-      core.info(`validated credentials`);
-      await exportAccountId(credentialsClient, maskAccountId);
-      core.info(`exported account id`);
+      if (outputEnvCredentials) {
+        await exportAccountId(credentialsClient, maskAccountId);
+      }
     } else {
       core.info('Proceeding with IAM user credentials');
     }