Adds Log() to MatchedRule, fixes audit log without log

[M4tteoP]

This PR proposes to enrich MatchedRule with Log() for filtering purposes.
It permits to:

• Filter the MatchedRules and obtain only the ones meant to be logged (indeed similar to what the original reporter (@joshi-mohit) tried to do with relying on Rule().Severity(), but that field was not suitable for this purpose).

	for _, matchedRule := range tx.MatchedRules() {
		if matchedRule.Log() {
		....
	}

Edit: with the workaround:

	for _, matchedRule := range tx.MatchedRules() {
		type ruleLogger interface{ Log() bool }
		mrWithlog, ok := matchedRule.(types.RuleLogger)
		if ok && mrWithlog.Log() {
			....
	}

• Do not alter MatchedRule: meant to internally collect any matched rule ( possible implications may be related to audit logs, Restricts MatchedRule list to rules with log directive #840 (comment) or anything else that adds an exception to the code may (or will) lead).

Supersedes #840
Fixes #839

[codecov]

Codecov Report

Patch coverage: 80.30% and project coverage change: +0.07 🎉

Comparison is base (1b8a007) 81.51% compared to head (42eedcd) 81.58%.

Additional details and impacted files

@@            Coverage Diff             @@
##             main     #848      +/-   ##
==========================================
+ Coverage   81.51%   81.58%   +0.07%     
==========================================
  Files         158      159       +1     
  Lines        8947     9010      +63     
==========================================
+ Hits         7293     7351      +58     
- Misses       1408     1412       +4     
- Partials      246      247       +1     

Flag	Coverage Δ
default	76.63% <82.17%> (+0.46%)	⬆️
examples	25.52% <16.66%> (+<0.01%)	⬆️
ftw	46.85% <40.47%> (-0.24%)	⬇️
ftw-multiphase	48.98% <40.47%> (-0.25%)	⬇️
tinygo	74.85% <96.59%> (+2.62%)	⬆️

Flags with carried forward coverage won't be shown. Click here to find out more.

Impacted Files	Coverage Δ
http/e2e/cmd/httpe2e/main.go	0.00% <ø> (ø)
internal/actions/allow.go	100.00% <ø> (ø)
internal/actions/auditlog.go	72.72% <ø> (ø)
internal/actions/block.go	100.00% <ø> (ø)
internal/actions/capture.go	100.00% <ø> (ø)
internal/actions/chain.go	100.00% <ø> (ø)
internal/actions/ctl.go	92.87% <ø> (ø)
internal/actions/deny.go	84.21% <ø> (ø)
internal/actions/drop.go	84.21% <ø> (ø)
internal/actions/exec.go	70.00% <ø> (ø)
... and 32 more

... and 1 file with indirect coverage changes

☔ View full report in Codecov by Sentry.
📢 Do you have feedback about the report comment? Let us know in this issue.

[jptosso]

#846 back to this

[jcchavezs]

Are there any assertions about the content of the logs?