diff --git a/cmd/HellPot/HellPot.go b/cmd/HellPot/HellPot.go index e2a81a5..d562f82 100644 --- a/cmd/HellPot/HellPot.go +++ b/cmd/HellPot/HellPot.go @@ -92,7 +92,6 @@ func main() { if runningConfig == nil { if runningConfig, err = config.Setup(nil); err != nil || runningConfig == nil { panic("failed to setup default config...\n" + err.Error()) - return // unreachable, but the linter doesn't seem to realize that } } diff --git a/internal/config/models_test.go b/internal/config/models_test.go index b7e9c51..c8b080f 100644 --- a/internal/config/models_test.go +++ b/internal/config/models_test.go @@ -29,19 +29,19 @@ func TestCompileRules(t *testing.T) { if len(rules.UseragentDisallowStrings) != 3 { t.Error("expected 3 got", len(rules.UseragentDisallowStrings)) } - if !rules.MatchUseragent("yeeterson") { + if !rules.MatchUseragent([]byte("yeeterson")) { t.Error("expected true got false") } - if !rules.MatchUseragent("mc") { + if !rules.MatchUseragent([]byte("mc")) { t.Error("expected true got false") } - if !rules.MatchUseragent("yooterson") { + if !rules.MatchUseragent([]byte("yooterson")) { t.Error("expected true got false") } - if !rules.MatchUseragent("gooters%n") { + if !rules.MatchUseragent([]byte("gooters%n")) { t.Error("expected true got false") } - if rules.MatchUseragent("yootersongooterson") { + if rules.MatchUseragent([]byte("yootersongooterson")) { t.Error("expected false got true") } }