removes toUpperCase from signature calculation (#10)

craftgate · Jan 20, 2021 · 98cf8cb · 98cf8cb
1 parent 386e558
commit 98cf8cb
Show file tree

Hide file tree

Showing 2 changed files with 1 addition and 2 deletions.
diff --git a/src/lib/utils.ts b/src/lib/utils.ts
@@ -22,7 +22,7 @@ export function generateRandomString(length = 6): string {
 export function calculateSignature({apiKey, secretKey, url, body, randomStr}: { apiKey: string; secretKey: string; url: string; randomStr: string; body?: string }): string {
   const hashStr: string = [url, apiKey, secretKey, randomStr, body].filter(s => !!s).join('');
   const hash = crypto.SHA256(hashStr);
-  return crypto.enc.Base64.stringify(hash).toUpperCase();
+  return crypto.enc.Base64.stringify(hash);
 }
 
 /**

diff --git a/test/lib/HttpClient.test.js b/test/lib/HttpClient.test.js
@@ -3,7 +3,6 @@ const MockAdapter = require('axios-mock-adapter');
 
 const {HttpClient} = require('../../dist/lib/HttpClient');
 const CraftgateError = require('../../dist').CraftgateError;
-const {calculateSignature} = require('../../dist/lib/utils');
 
 test('HttpClient should translate API errors to CraftgateError errors when status code is 200', async t => {
   const client = new HttpClient({