fix: skip whitespace after encountering media type separator without …

…parameters
DASPRiD · Jul 1, 2024 · 16bd1a3 · 16bd1a3
1 parent 70cf7b2
commit 16bd1a3
Show file tree

Hide file tree

Showing 2 changed files with 9 additions and 0 deletions.
diff --git a/src/accept.ts b/src/accept.ts
@@ -78,6 +78,7 @@ export class AcceptParser {
         }
 
         if (this.readSeparator() === ",") {
+            this.skipWhitespace();
             return [{ ...AcceptParser.default, type, subType }, true];
         }
 

diff --git a/test/accept.test.ts b/test/accept.test.ts
@@ -38,6 +38,14 @@ describe("AcceptParser", () => {
                 { type: "application", subType: "xml", parameters: {}, weight: 1, acceptExt: {} },
             ],
         ],
+        [
+            "multiple types with spaces",
+            "application/json , application/xml",
+            [
+                { type: "application", subType: "json", parameters: {}, weight: 1, acceptExt: {} },
+                { type: "application", subType: "xml", parameters: {}, weight: 1, acceptExt: {} },
+            ],
+        ],
         [
             "type with parameter",
             "application/json ; foo=bar",
-Original file line number
+Diff line change
@@ Expand Up / @@ -78,6 +78,7 @@ export class AcceptParser { @@
             }
             if (this.readSeparator() === ",") {
+                this.skipWhitespace();
                 return [{ ...AcceptParser.default, type, subType }, true];
             }
@@ Expand Down @@