Fixing linter

Signed-off-by: Harsha Vamsi Kalluri <[email protected]>
opensearch-project · Feb 21, 2023 · 565ec7b · 565ec7b
1 parent ba2a2f6
commit 565ec7b
Show file tree

Hide file tree

Showing 2 changed files with 55 additions and 55 deletions.
diff --git a/lib/aws/AwsSigv4Signer.js b/lib/aws/AwsSigv4Signer.js
@@ -18,67 +18,67 @@ const AwsSigv4SignerError = require('./errors')
 const getAwsSDKCredentialsProvider = async () => {
   // First try V3
   try {
-    const awsV3 = await import('@aws-sdk/credential-provider-node');
+    const awsV3 = await import('@aws-sdk/credential-provider-node')
     if (typeof awsV3.defaultProvider === 'function') {
-      return awsV3.defaultProvider();
+      return awsV3.defaultProvider()
     }
   } catch (err) {
     // Ignore
   }
   try {
-    const awsV2 = await import('aws-sdk');
+    const awsV2 = await import('aws-sdk')
     if (awsV2.default && typeof awsV2.default.config.getCredentials === 'function') {
       return () =>
         new Promise((resolve, reject) => {
           awsV2.default.config.getCredentials((err, credentials) => {
             if (err) {
-              reject(err);
+              reject(err)
             } else {
-              resolve(credentials);
+              resolve(credentials)
             }
-          });
-        });
+          })
+        })
     }
   } catch (err) {
     // Ignore
   }
 
   throw new AwsSigv4SignerError(
     'Unable to find a valid AWS SDK, please provide a valid getCredentials function to AwsSigv4Signer options.'
-  );
-};
+  )
+}
 
 const awsDefaultCredentialsProvider = () =>
   new Promise((resolve, reject) => {
     getAwsSDKCredentialsProvider()
       .then((provider) => {
-        provider().then(resolve).catch(reject);
+        provider().then(resolve).catch(reject)
       })
       .catch((err) => {
-        reject(err);
-      });
-  });
+        reject(err)
+      })
+  })
 
-function AwsSigv4Signer(opts = {}) {
+function AwsSigv4Signer (opts = {}) {
   const credentialsState = {
-    credentials: null,
-  };
+    credentials: null
+  }
   if (!opts.region) {
-    throw new AwsSigv4SignerError('Region cannot be empty');
+    throw new AwsSigv4SignerError('Region cannot be empty')
   }
   if (!opts.service) {
-    opts.service = 'es';
+    opts.service = 'es'
   }
   if (typeof opts.getCredentials !== 'function') {
-    opts.getCredentials = awsDefaultCredentialsProvider;
+    opts.getCredentials = awsDefaultCredentialsProvider
   }
 
-  function buildSignedRequestObject(request = {}) {
-    request.service = opts.service;
-    request.region = opts.region;
-    request.headers = request.headers || {};
-    request.headers['host'] = request.hostname;
-    return aws4.sign(request, credentialsState.credentials);
+  function buildSignedRequestObject (request = {}) {
+    request.service = opts.service
+    request.region = opts.region
+    request.headers = request.headers || {}
+    request.headers.host = request.hostname
+    return aws4.sign(request, credentialsState.credentials)
   }
 
   class AwsSigv4SignerConnection extends Connection {

diff --git a/test/unit/lib/aws/awssigv4signer.test.js b/test/unit/lib/aws/awssigv4signer.test.js
@@ -17,7 +17,7 @@ const { Connection } = require('../../../../index')
 const { Client, buildServer } = require('../../../utils')
 
 test('Sign with SigV4', (t) => {
-  t.plan(3);
+  t.plan(3)
 
   const mockCreds = {
     accessKeyId: uuidv4(),
@@ -48,19 +48,19 @@ test('Sign with SigV4', (t) => {
     }
   })
 
-  const signedRequest = auth.buildSignedRequestObject(request);
-  t.hasProp(signedRequest.headers, 'X-Amz-Date');
-  t.hasProp(signedRequest.headers, 'Authorization');
-  t.same(signedRequest.service, 'es');
-});
+  const signedRequest = auth.buildSignedRequestObject(request)
+  t.hasProp(signedRequest.headers, 'X-Amz-Date')
+  t.hasProp(signedRequest.headers, 'Authorization')
+  t.same(signedRequest.service, 'es')
+})
 
 test('Sign with SigV4 failure (with empty region)', (t) => {
   const mockCreds = {
     accessKeyId: uuidv4(),
     secretAccessKey: uuidv4()
   }
 
-  const mockRegions = [{ region: undefined }, { region: null }, { region: '' }, {}];
+  const mockRegions = [{ region: undefined }, { region: null }, { region: '' }, {}]
 
   const AwsSigv4SignerOptions = {
     getCredentials: () =>
@@ -71,51 +71,51 @@ test('Sign with SigV4 failure (with empty region)', (t) => {
 
   mockRegions.forEach((region) => {
     try {
-      AwsSigv4Signer(Object.assign({}, AwsSigv4SignerOptions, region));
-      t.fail('Should fail');
+      AwsSigv4Signer(Object.assign({}, AwsSigv4SignerOptions, region))
+      t.fail('Should fail')
     } catch (err) {
-      t.ok(err instanceof AwsSigv4SignerError);
-      t.same(err.message, 'Region cannot be empty');
+      t.ok(err instanceof AwsSigv4SignerError)
+      t.same(err.message, 'Region cannot be empty')
     }
-  });
+  })
 
-  t.end();
-});
+  t.end()
+})
 
 test('Sign with SigV4 and provided service', (t) => {
-  t.plan(1);
+  t.plan(1)
 
   const mockCreds = {
     accessKeyId: uuidv4(),
-    secretAccessKey: uuidv4(),
-  };
+    secretAccessKey: uuidv4()
+  }
 
-  const mockRegion = 'us-west-2';
-  const mockService = 'foo';
+  const mockRegion = 'us-west-2'
+  const mockService = 'foo'
 
   const AwsSigv4SignerOptions = {
     getCredentials: () =>
       new Promise((resolve) => {
-        setTimeout(() => resolve(mockCreds), 100);
+        setTimeout(() => resolve(mockCreds), 100)
       }),
     region: mockRegion,
-    service: mockService,
-  };
+    service: mockService
+  }
 
-  const auth = AwsSigv4Signer(AwsSigv4SignerOptions);
+  const auth = AwsSigv4Signer(AwsSigv4SignerOptions)
 
   const connection = new Connection({
-    url: new URL('https://localhost:9200'),
-  });
+    url: new URL('https://localhost:9200')
+  })
 
   const request = connection.buildRequestObject({
     path: '/hello',
-    method: 'GET',
-  });
+    method: 'GET'
+  })
 
-  const signedRequest = auth.buildSignedRequestObject(request);
-  t.same(signedRequest.service, mockService);
-});
+  const signedRequest = auth.buildSignedRequestObject(request)
+  t.same(signedRequest.service, mockService)
+})
 
 test('Sign with SigV4 failure (without getCredentials function)', (t) => {
   t.plan(2)