(v3.x) perf: only check for isEssence once in RegExp for content-type…

…-parser (#4481) (#4497)
fastify · Jan 6, 2023 · e4e8393 · e4e8393
1 parent c172cae
commit e4e8393
Showing 1 changed file with 2 additions and 1 deletion.
diff --git a/lib/contentTypeParser.js b/lib/contentTypeParser.js
@@ -72,6 +72,7 @@ ContentTypeParser.prototype.add = function (contentType, opts, parserFn) {
     if (contentTypeIsString) {
       this.parserList.unshift(new ParserListItem(contentType))
     } else {
+      contentType.isEssence = contentType.source.indexOf(';') === -1
       this.parserRegExpList.unshift(contentType)
     }
     this.customParsers.set(contentType.toString(), parser)
@@ -396,7 +397,7 @@ function compareContentType (contentType, parserListItem) {
 }
 
 function compareRegExpContentType (contentType, essenceMIMEType, regexp) {
-  if (regexp.source.indexOf(';') === -1) {
+  if (regexp.isEssence) {
     // we do essence check
     return regexp.test(essenceMIMEType)
   } else {