chore: replace Promise resolve/reject with async

Author: trivikr

packages/credential-provider-imds/src/fromContainerMetadata.ts

@@ -19,9 +19,9 @@ export const ENV_CMDS_AUTH_TOKEN = "AWS_CONTAINER_AUTHORIZATION_TOKEN";
 export const fromContainerMetadata = (init: RemoteProviderInit = {}): CredentialProvider => {
   const { timeout, maxRetries } = providerConfigFromInit(init);
   return async () => {
-    const url = await getCmdsUri();
+    const requestOptions = await getCmdsUri();
     return await retry(async () => {
-      const credsResponse = JSON.parse(await requestFromEcsImds(timeout, url));
+      const credsResponse = JSON.parse(await requestFromEcsImds(timeout, requestOptions));
       if (!isImdsCredentials(credsResponse)) {
         throw new ProviderError("Invalid response received from instance metadata service.");
       }
@@ -32,9 +32,10 @@ export const fromContainerMetadata = (init: RemoteProviderInit = {}): Credential
 
 const requestFromEcsImds = async (timeout: number, options: RequestOptions): Promise<string> => {
   if (process.env[ENV_CMDS_AUTH_TOKEN]) {
-    const { headers = {} } = options;
-    headers.Authorization = process.env[ENV_CMDS_AUTH_TOKEN];
-    options.headers = headers;
+    options.headers = {
+      ...options.headers,
+      Authorization: process.env[ENV_CMDS_AUTH_TOKEN],
+    };
   }
 
   const buffer = await httpRequest({
@@ -54,40 +55,34 @@ const GREENGRASS_PROTOCOLS = {
   "https:": true,
 };
 
-const getCmdsUri = (): Promise<RequestOptions> => {
+const getCmdsUri = async (): Promise<RequestOptions> => {
   if (process.env[ENV_CMDS_RELATIVE_URI]) {
-    return Promise.resolve({
+    return {
       hostname: CMDS_IP,
       path: process.env[ENV_CMDS_RELATIVE_URI],
-    });
+    };
   }
 
   if (process.env[ENV_CMDS_FULL_URI]) {
     const parsed = parse(process.env[ENV_CMDS_FULL_URI]!);
     if (!parsed.hostname || !(parsed.hostname in GREENGRASS_HOSTS)) {
-      return Promise.reject(
-        new ProviderError(`${parsed.hostname} is not a valid container metadata service hostname`, false)
-      );
+      throw new ProviderError(`${parsed.hostname} is not a valid container metadata service hostname`, false);
     }
 
     if (!parsed.protocol || !(parsed.protocol in GREENGRASS_PROTOCOLS)) {
-      return Promise.reject(
-        new ProviderError(`${parsed.protocol} is not a valid container metadata service protocol`, false)
-      );
+      throw new ProviderError(`${parsed.protocol} is not a valid container metadata service protocol`, false);
     }
 
-    return Promise.resolve({
+    return {
       ...parsed,
       port: parsed.port ? parseInt(parsed.port, 10) : undefined,
-    });
+    };
   }
 
-  return Promise.reject(
-    new ProviderError(
-      "The container metadata credential provider cannot be used unless" +
-        ` the ${ENV_CMDS_RELATIVE_URI} or ${ENV_CMDS_FULL_URI} environment` +
-        " variable is set",
-      false
-    )
+  throw new ProviderError(
+    "The container metadata credential provider cannot be used unless" +
+      ` the ${ENV_CMDS_RELATIVE_URI} or ${ENV_CMDS_FULL_URI} environment` +
+      " variable is set",
+    false
   );
 };