[fix] Always configure the engine with the default detectors (#3152)

If detectors are not wanted by a user, they can be filtered out via
the `--include-detectors` or `--exclude-detectors` flag.
This commit is contained in:
Miccah 2024-08-02 07:48:39 -07:00 committed by GitHub
parent 05e4635824
commit eccbca730d
No known key found for this signature in database
GPG key ID: B5690EEEBB952194

View file

@ -465,8 +465,12 @@ func run(state overseer.State) {
}
engConf := engine.Config{
Concurrency: *concurrency,
Detectors: conf.Detectors,
Concurrency: *concurrency,
// The engine must always be configured with the list of
// default detectors, which can be further filtered by the
// user. The filters are applied by the engine and are only
// subtractive.
Detectors: append(engine.DefaultDetectors(), conf.Detectors...),
Verify: !*noVerification,
IncludeDetectors: *includeDetectors,
ExcludeDetectors: *excludeDetectors,