refact cmd/crowdsec: encapsulate cache into alertBuffer#4300
Conversation
|
@mmetc: There are no area labels on this PR. You can add as many areas as you see fit.
DetailsI 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. |
Codecov Report❌ Patch coverage is
Additional details and impacted files@@ Coverage Diff @@
## master #4300 +/- ##
==========================================
- Coverage 63.28% 63.26% -0.03%
==========================================
Files 472 472
Lines 33382 33384 +2
==========================================
- Hits 21126 21120 -6
- Misses 10142 10151 +9
+ Partials 2114 2113 -1
Flags with carried forward coverage won't be shown. Click here to find out more. ☔ View full report in Codecov by Sentry. 🚀 New features to boost your workflow:
|
simplify loop and reduce risk of races (we didn't lock before reading len)
e01dcd1 to
128796b
Compare
simplify loop and reduce risk of races (we didn't lock before reading len)