Merge pull request #2368 from asger-semmle/regexp-max-length

Approved by max-schaefer

Author: semmle-qlci

javascript/extractor/src/com/semmle/js/extractor/ASTExtractor.java

@@ -520,7 +520,7 @@ public Label visit(Literal nd, Context c) {
         OffsetTranslation offsets = new OffsetTranslation();
         offsets.set(0, 1); // skip the initial '/'
         regexpExtractor.extract(source.substring(1, source.lastIndexOf('/')), offsets, nd, false);
-      } else if (nd.isStringLiteral() && !c.isInsideType()) {
+      } else if (nd.isStringLiteral() && !c.isInsideType() && nd.getRaw().length() < 1000) {
         regexpExtractor.extract(valueString, makeStringLiteralOffsets(nd.getRaw()), nd, true);
       }
       return key;

javascript/extractor/src/com/semmle/js/extractor/Main.java

@@ -37,7 +37,7 @@ public class Main {
    * A version identifier that should be updated every time the extractor changes in such a way that
    * it may produce different tuples for the same file under the same {@link ExtractorConfig}.
    */
-  public static final String EXTRACTOR_VERSION = "2019-10-23";
+  public static final String EXTRACTOR_VERSION = "2019-11-17";
 
   public static final Pattern NEWLINE = Pattern.compile("\n");