fix: support multipart upload boundaryId with quotes

jack-michaud · jack-michaud · commit ce79d4cc1a11 · 2022-04-08T13:14:48.000-07:00
diff --git a/src/emulator/storage/multipart.ts b/src/emulator/storage/multipart.ts
@@ -125,7 +125,11 @@ export function parseObjectUploadMultipartRequest(
   if (!contentTypeHeader.startsWith("multipart/related")) {
     throw new Error(`Invalid Content-Type: ${contentTypeHeader}`);
   }
-  const boundaryId = contentTypeHeader.split("boundary=")[1];
+  let boundaryId = contentTypeHeader.split("boundary=")[1];
+
+  // fix for supporting boundaryId with quotes
+  boundaryId = boundaryId.replace(/"/g, "");
+
   if (!boundaryId) {
     throw new Error(`Invalid Content-Type header: ${contentTypeHeader}`);
   }

Original file line number	Diff line number	Diff line change
`@@ -125,7 +125,11 @@ export function parseObjectUploadMultipartRequest(`
`125`	`125`	`if (!contentTypeHeader.startsWith("multipart/related")) {`
`126`	`126`	throw new Error(`Invalid Content-Type: ${contentTypeHeader}`);
`127`	`127`	`}`
`128`		`- const boundaryId = contentTypeHeader.split("boundary=")[1];`
	`128`	`+ let boundaryId = contentTypeHeader.split("boundary=")[1];`
	`129`	`+`
	`130`	`+ // fix for supporting boundaryId with quotes`
	`131`	`+ boundaryId = boundaryId.replace(/"/g, "");`
	`132`	`+`
`129`	`133`	`if (!boundaryId) {`
`130`	`134`	throw new Error(`Invalid Content-Type header: ${contentTypeHeader}`);
`131`	`135`	`}`