fix(Scenarios/XForwardedFor): correct the trigger of the alert + add cache to skip double ip extraction by thib3113 · Pull Request #30 · thib3113/node-crowdsec

thib3113 · 2023-08-20T01:16:37Z

correcting the way the alert is trigger + add cache to skip double ip extraction

- correct the trigger of the alert - add cache to skip double ip extraction

codecov-commenter · 2023-08-20T01:19:26Z

Codecov Report

Merging #30 (a53cc14) into main (0700b0d) will increase coverage by 0.06%.
The diff coverage is 61.53%.

@@            Coverage Diff             @@
##             main      #30      +/-   ##
==========================================
+ Coverage   58.19%   58.25%   +0.06%     
==========================================
  Files          30       30              
  Lines         952      963      +11     
  Branches      194      195       +1     
==========================================
+ Hits          554      561       +7     
- Misses        350      354       +4     
  Partials       48       48

Flag	Coverage Δ
crowdsec-client	`100.00% <ø> (ø)`
crowdsec-client-scenarios	`37.75% <61.53%> (+1.23%)`	⬆️
crowdsec-http-middleware	`0.00% <ø> (ø)`

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

Files Changed	Coverage Δ
...rc/scenarios/XForwardedFor/XForwardedForChecker.ts	`78.12% <61.53%> (-1.88%)`	⬇️

Continue to review full report in Codecov by Sentry.

Legend - Click here to learn more
Δ = absolute <relative> (impact), ø = not affected, ? = missing data
Powered by Codecov. Last update 0700b0d...a53cc14. Read the comment docs.