Merge 247978f into 648e456

thoughtworks · Sep 7, 2020 · edf9867 · edf9867
2 parents 648e456 + 247978f
commit edf9867
Show file tree

Hide file tree

Showing 4 changed files with 12 additions and 256 deletions.
diff --git a/.gitignore b/.gitignore
@@ -15,3 +15,4 @@ detector/.talismanrc
 coverage.out
 coverage.txt
 .talismanrc
+talisman_reports/
diff --git a/detector/pattern/pattern_detector.go b/detector/pattern/pattern_detector.go
@@ -75,9 +75,18 @@ func (detector PatternDetector) Test(comparator helpers.ChecksumCompare, current
 	}
 }
 
+var knownPatterns = make(map[string]*regexp.Regexp)
+
 func replaceAllStrings(str string, pattern string) string {
-	re := regexp.MustCompile(fmt.Sprintf("(?i)%s", pattern))
-	return re.ReplaceAllString(str, "")
+	var pat *regexp.Regexp
+	var ok bool
+
+	if pat, ok = knownPatterns[pattern]; !ok {
+		pat = regexp.MustCompile(fmt.Sprintf("(?i)%s", pattern))
+		knownPatterns[pattern] = pat
+	}
+
+	return pat.ReplaceAllString(str, "")
 }
 
 func processAllowedPatterns(addition gitrepo.Addition, tRC *talismanrc.TalismanRC) string {

diff --git a/talisman_reports/report.html b/talisman_reports/report.html
diff --git a/talisman_reports/report.json b/talisman_reports/report.json