appsec: do not attempt to deduplicate native modsec rules

[buixor]

Do not attempt to dedup modsec native rules as they might have no IDs. Doing this can mess up stuff (see #3343 and #3350 )

[github-actions]

@buixor: There are no 'kind' label on this PR. You need a 'kind' label to generate the release automatically.

• /kind feature
• /kind enhancement
• /kind refactoring
• /kind fix
• /kind chore
• /kind dependencies

Details

I am a bot created to help the crowdsecurity developers manage community feedback and contributions. You can check out my manifest file to understand my behavior and what I can do. If you want to use this for your project, you can check out the BirthdayResearch/oss-governance-bot repository.

[github-actions]

@buixor: There are no area labels on this PR. You can add as many areas as you see fit.

• /area agent
• /area local-api
• /area cscli
• /area appsec
• /area security
• /area configuration

Details

I am a bot created to help the crowdsecurity developers manage community feedback and contributions. You can check out my manifest file to understand my behavior and what I can do. If you want to use this for your project, you can check out the BirthdayResearch/oss-governance-bot repository.

[buixor]

/area agent
/kind fix

[codecov]

Codecov Report

Attention: Patch coverage is 84.61538% with 2 lines in your changes missing coverage. Please review.

Project coverage is 59.31%. Comparing base (4935dc5) to head (41177a3).
Report is 3 commits behind head on master.

Files with missing lines	Patch %	Lines
pkg/appsec/appsec_rules_collection.go	0.00%	2 Missing ⚠️

Additional details and impacted files

@@           Coverage Diff           @@
##           master    #3347   +/-   ##
=======================================
  Coverage   59.30%   59.31%           
=======================================
  Files         358      358           
  Lines       38433    38428    -5     
=======================================
- Hits        22794    22793    -1     
+ Misses      13744    13739    -5     
- Partials     1895     1896    +1     

Flag	Coverage Δ
bats	42.24% <0.00%> (+0.04%)	⬆️
unit-linux	33.22% <84.61%> (+0.02%)	⬆️
unit-windows	28.49% <84.61%> (+<0.01%)	⬆️

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

☔ View full report in Codecov by Sentry.
📢 Have feedback on the report? Share it here.

[sabban]

lgtm