add rateLimitExtraTimeout option. code style fixes

Author: ThisNils

resources/structs.ts

@@ -11,9 +11,19 @@ export interface ClientConfig {
   apiKey?: string;
 
   /**
-   * The default language for all endpoints. Defaults to 'en'
+   * The default language for all endpoints. Defaults to `en`
    */
   language: Language;
+
+  /**
+   * Extra timeout for stats ratelimits. Defaults to `0`.
+   *
+   * Normally the client will send 3 stats requests per 1100 milliseconds.
+   * Setting this option to `100` increases the rate to 3 per 1200 milliseconds.
+   *
+   * You should increase this option if you're getting 429 responses
+   */
+  rateLimitExtraTimeout: number;
 }
 
 export interface ClientOptions extends Partial<ClientConfig> {}

src/client/Client.ts

@@ -21,6 +21,7 @@ class Client {
   constructor(config?: ClientOptions) {
     this.config = {
       language: Language.English,
+      rateLimitExtraTimeout: 0,
       ...config,
     };
 

src/http/HTTP.ts

@@ -1,12 +1,12 @@
 /* eslint-disable no-restricted-syntax */
 import axios, { AxiosError, AxiosInstance } from 'axios';
 import rateLimit, { RateLimitedAxiosInstance } from 'axios-rate-limit';
-import { URLSearchParams } from 'url';
 import { version } from '../../package.json';
 import Client from '../client/Client';
 import FortniteAPIError from '../exceptions/FortniteAPIError';
 import InvalidAPIKeyError from '../exceptions/InvalidAPIKeyError';
 import MissingAPIKeyError from '../exceptions/MissingAPIKeyError';
+import { serializeParams } from '../util/util';
 import { FortniteAPIResponseData } from './httpStructs';
 
 class HTTP {
@@ -29,27 +29,18 @@ class HTTP {
       },
     });
 
-    this.statsAxios = rateLimit(this.axios, { maxRequests: 3, perMilliseconds: 1100 });
+    this.statsAxios = rateLimit(this.axios, {
+      maxRequests: 3,
+      perMilliseconds: 1100 + this.client.config.rateLimitExtraTimeout,
+    });
   }
 
   public async fetch(url: string, params?: any): Promise<FortniteAPIResponseData> {
     try {
       const response = await this.axios({
         url,
         params,
-        paramsSerializer: (p) => {
-          const searchParams = new URLSearchParams();
-
-          for (const [key, value] of Object.entries(p)) {
-            if (Array.isArray(value)) {
-              for (const singleValue of value) searchParams.append(key, singleValue);
-            } else {
-              searchParams.append(key, (value as any));
-            }
-          }
-
-          return searchParams.toString();
-        },
+        paramsSerializer: serializeParams,
       });
 
       return response.data;
@@ -72,22 +63,12 @@ class HTTP {
 
   public async fetchStats(url: string, params?: any): Promise<FortniteAPIResponseData> {
     try {
-      const response = await this.statsAxios.get(url, {
+      const response = await this.statsAxios({
+        url,
         params,
-        paramsSerializer: (p) => {
-          const searchParams = new URLSearchParams();
-
-          for (const [key, value] of Object.entries(p)) {
-            if (Array.isArray(value)) {
-              for (const singleValue of value) searchParams.append(key, singleValue);
-            } else {
-              searchParams.append(key, (value as any));
-            }
-          }
-
-          return searchParams.toString();
-        },
+        paramsSerializer: serializeParams,
       });
+
       return response.data;
     } catch (e) {
       if (e instanceof AxiosError && e.response?.data?.error) {

src/util/util.ts

@@ -0,0 +1,17 @@
+/* eslint-disable no-restricted-syntax */
+/* eslint-disable import/prefer-default-export */
+import { URLSearchParams } from 'url';
+
+export const serializeParams = (params: any) => {
+  const searchParams = new URLSearchParams();
+
+  for (const [key, value] of Object.entries(params)) {
+    if (Array.isArray(value)) {
+      for (const singleValue of value) searchParams.append(key, singleValue);
+    } else {
+      searchParams.append(key, (value as any));
+    }
+  }
+
+  return searchParams.toString();
+};