Do not support the s flag

tjenkinson · Oct 21, 2023 · 8ed3676 · 8ed3676
1 parent a6d0ed7
commit 8ed3676
Show file tree

Hide file tree

Showing 2 changed files with 2 additions and 8 deletions.
diff --git a/src/redos-detector.test.ts b/src/redos-detector.test.ts
@@ -599,7 +599,7 @@ describe('RedosDetector', () => {
       expect(() => isSafe(/a/m)).toThrowError('Unsupported flag: m');
     });
 
-    ['u', 'g', 's', 'y', 'i'].forEach((flag) => {
+    ['u', 'g', 'y', 'i'].forEach((flag) => {
       it(`supports the "${flag}" flag`, () => {
         expect(() => isSafe(new RegExp('a', flag))).not.toThrowError();
       });

diff --git a/src/redos-detector.ts b/src/redos-detector.ts
@@ -226,13 +226,7 @@ export const defaultMaxBacktracks = 200;
 export const defaultMaxSteps = 20000;
 export const defaultUnicode = false;
 export const defaultCaseInsensitive = false;
-const supportedJSFlags: ReadonlySet<string> = new Set([
-  'u',
-  'g',
-  's',
-  'y',
-  'i',
-]);
+const supportedJSFlags: ReadonlySet<string> = new Set(['u', 'g', 'y', 'i']);
 
 type PatternWithAtomicGroupOffsets = Readonly<{
   atomicGroupOffsets: ReadonlySet<number>;