fatih-acar commented on code in PR #7741:
URL: https://github.com/apache/trafficcontrol/pull/7741#discussion_r1316219342
##########
cache-config/t3c-check-refs/t3c-check-refs.go:
##########
@@ -115,7 +115,7 @@ func checkConfigLine(line string, lineNumber int,
filesAdding map[string]struct{
// are assumed to be configuration files and
are checked that they
// exist in the filesystem at the absolute
location in the name
// or relative to the ATS configuration files
directory.
- m :=
regexp.MustCompile(`^*(\.config|\.cfg|\.txt|\.yml|\.yaml|\.lua)+`)
+ m :=
regexp.MustCompile(`^*(\.config|\.cfg|\.txt|\.yml|\.yaml|\.lua)$`)
Review Comment:
Indeed, the beginning of the regex is not valid too and it seems the Go
regexp lib is not complaining
I have fixed that too with your suggestion, thanks
--
This is an automated message from the Apache Git Service.
To respond to the message, please log on to GitHub and use the
URL above to go to the specific comment.
To unsubscribe, e-mail: [email protected]
For queries about this service, please contact Infrastructure at:
[email protected]
