⚡ Improve list of content-types to download

n8n-io · Oct 20, 2022 · 22d4819 · 22d4819
1 parent 8f14baf
commit 22d4819
Show file tree

Hide file tree

Showing 2 changed files with 32 additions and 2 deletions.
diff --git a/packages/nodes-base/nodes/HttpRequest/GenericFunctions.ts b/packages/nodes-base/nodes/HttpRequest/GenericFunctions.ts
@@ -68,3 +68,29 @@ export const getOAuth2AdditionalParameters = (nodeCredentialType: string) => {
 	};
 	return oAuth2Options[nodeCredentialType];
 };
+
+//https://developer.mozilla.org/en-US/docs/Web/HTTP/Basics_of_HTTP/MIME_types/Common_types
+export const binaryContentTypes = [
+	'image/',
+	'audio/',
+	'video/',
+	'application/octet-stream',
+	'application/gzip',
+	'application/zip',
+	'application/vnd.rar',
+	'application/epub+zip',
+	'application/x-bzip',
+	'application/x-bzip2',
+	'application/x-cdf',
+	'application/vnd.amazon.ebook',
+	'application/msword',
+	'application/vnd.openxmlformats-officedocument.wordprocessingml.document',
+	'application/vnd.ms-fontobject',
+	'application/vnd.oasis.opendocument.presentation',
+	'application/pdf',
+	'application/x-tar',
+	'application/vnd.visio',
+	'application/vnd.ms-excel',
+	'application/vnd.openxmlformats-officedocument.spreadsheetml.sheet',
+	'application/x-7z-compressed',
+];
diff --git a/packages/nodes-base/nodes/HttpRequest/V3/HttpRequestV3.node.ts b/packages/nodes-base/nodes/HttpRequest/V3/HttpRequestV3.node.ts
@@ -12,7 +12,11 @@ import {
 
 import { OptionsWithUri } from 'request-promise-native';
 
-import { getOAuth2AdditionalParameters, replaceNullValues } from '../GenericFunctions';
+import {
+	binaryContentTypes,
+	getOAuth2AdditionalParameters,
+	replaceNullValues,
+} from '../GenericFunctions';
 export class HttpRequestV3 implements INodeType {
 	description: INodeTypeDescription;
 
@@ -1208,7 +1212,7 @@ export class HttpRequestV3 implements INodeType {
 				if (responseContentType.includes('application/json')) {
 					responseFormat = 'json';
 					response.body = JSON.parse(Buffer.from(response.body).toString());
-				} else if (['image/', 'audio/', 'video/', 'application/octet-stream'].some((e) => responseContentType.includes(e))) {
+				} else if (binaryContentTypes.some((e) => responseContentType.includes(e))) {
 					responseFormat = 'file';
 				} else {
 					responseFormat = 'text';