Merge pull request #14 from tiagosiebler/#12/fixgetsign

fix(#12): fix sign error for some get requests
tiagosiebler · May 7, 2021 · 0ad9b80 · 0ad9b80
2 parents 7dc968d + 44e694f
commit 0ad9b80
Show file tree

Hide file tree

Showing 3 changed files with 4 additions and 4 deletions.
diff --git a/package-lock.json b/package-lock.json
diff --git a/package.json b/package.json
@@ -1,6 +1,6 @@
 {
   "name": "ftx-api",
-  "version": "1.0.8",
+  "version": "1.0.9",
   "description": "Node.js connector for FTX's REST APIs and WebSockets",
   "main": "lib/index.js",
   "types": "lib/index.d.ts",

diff --git a/src/util/requestWrapper.ts b/src/util/requestWrapper.ts
@@ -177,8 +177,8 @@ export default class RequestUtil {
   private getRequestSignature(
     method: Method,
     endpoint: string,
-    secret?: string | undefined,
-    params?: string | object
+    secret: string | undefined,
+    params: string | object = ''
   ): { timestamp: number; sign: string; } {
     const timestamp = Date.now() + (this.timeOffset || 0);
     if (!secret) {