Ignore no requests if ignoreSuffix is empty (#94)

An empty ignore suffix currently causes all requests to be ignored. This makes it impossible to disable ignoring by suffix.
apache · Oct 7, 2022 · f838ffc · f838ffc
1 parent 7232663
commit f838ffc
Showing 1 changed file with 13 additions and 4 deletions.
diff --git a/src/config/AgentConfig.ts b/src/config/AgentConfig.ts
@@ -53,10 +53,19 @@ export function finalizeConfig(config: AgentConfig): void {
     'i',
   );
 
-  const ignoreSuffix = `^.+(?:${config
-    .ignoreSuffix!.split(',')
-    .map((s) => escapeRegExp(s.trim()))
-    .join('|')})$`;
+  const convertIgnoreSuffix = (configuredIgnoreSuffix: string | undefined) => {
+    if (!configuredIgnoreSuffix) {
+      // This regexp will never match => no files are ignored.
+      return '\\A(?!x)x';
+    } else {
+      return `^.+(?:${configuredIgnoreSuffix!
+        .split(',')
+        .map((s) => escapeRegExp(s.trim()))
+        .join('|')})$`;
+    }
+  };
+
+  const ignoreSuffix = convertIgnoreSuffix(config.ignoreSuffix);
   const ignorePath =
     '^(?:' +
     config