Merge pull request #1065 from JupiterOne/http-client-text-body

http-client: add option to pass text body
JupiterOne · Apr 24, 2024 · 2e384ee · 2e384ee
2 parents 4a9d688 + 1690dc8
commit 2e384ee
Show file tree

Hide file tree

Showing 2 changed files with 11 additions and 3 deletions.
diff --git a/packages/integration-sdk-http-client/src/client.ts b/packages/integration-sdk-http-client/src/client.ts
@@ -241,6 +241,8 @@ export abstract class BaseAPIClient {
           }
           (fmtBody as FormData).append(key, value);
         });
+      } else if (bodyType === 'text' && typeof body === 'string') {
+        fmtBody = body;
       } else {
         fmtBody = JSON.stringify(body);
       }
@@ -250,6 +252,7 @@ export abstract class BaseAPIClient {
       headers: {
         ...(bodyType === 'json' && { 'Content-Type': 'application/json' }),
         ...(bodyType === 'form' && (fmtBody as FormData).getHeaders()),
+        ...(bodyType === 'text' && { 'Content-Type': 'text/plain' }),
         Accept: 'application/json',
         ...(authorize && this.authorizationHeaders),
         ...headers,
@@ -497,7 +500,12 @@ export abstract class BaseAPIClient {
         this.rateLimitThrottling.rateLimitHeaders?.reset ?? 'ratelimit-reset';
 
       this.logger.warn(
-        { rateLimitLimit, rateLimitRemaining, timeToSleepInMs },
+        {
+          endpoint: response.url,
+          rateLimitLimit,
+          rateLimitRemaining,
+          timeToSleepInMs,
+        },
         `Exceeded ${thresholdPercentage}% of rate limit. Sleeping until ${resetHeaderName}`,
       );
       await sleep(timeToSleepInMs);

diff --git a/packages/integration-sdk-http-client/src/types.ts b/packages/integration-sdk-http-client/src/types.ts
@@ -5,8 +5,8 @@ export type OptionalPromise<T> = T | Promise<T>;
 
 export interface RequestOptions {
   method?: 'GET' | 'POST';
-  body?: Record<string, unknown>;
-  bodyType?: 'json' | 'form';
+  body?: Record<string, unknown> | string;
+  bodyType?: 'json' | 'form' | 'text';
   headers?: Record<string, string>;
   authorize?: boolean;
   bucketTokens?: number;