Repository navigation
Fix. SFW. AC/AF pass cookie value fixed. - #917
Conversation
Codecov Report❌ Patch coverage is
❌ Your patch check has failed because the patch coverage (37.25%) is below the target coverage (70.00%). You can increase the patch coverage or adjust the target coverage. Additional details and impacted files@@ Coverage Diff @@
## dev #917 +/- ##
============================================
- Coverage 34.54% 34.54% -0.01%
- Complexity 7066 7071 +5
============================================
Files 298 298
Lines 27427 27468 +41
============================================
+ Hits 9476 9488 +12
- Misses 17951 17980 +29 ☔ View full report in Codecov by Harness. 🚀 New features to boost your workflow:
|
There was a problem hiding this comment.
🟢 Approval recommended
Production cookie generation and validation use the new format consistently, with no blocking issues identified.
No open issues 🎉
What changed in this PR
This PR hardens SpamFireWall pass cookies by including the pass/whitelist status in the hash and using a consistent format across cookie creation and validation.
Changes:
- Adds shared helpers to generate and strictly validate status-signed pass cookies.
- Updates firewall and AC/AF cookie issuance to use the shared generator.
- Updates related test code; test files were excluded from review.
| File | Description |
|---|---|
tests/ApbctWP/Firewall/TestSFW.php |
Updates pass-cookie tests; excluded from review. |
lib/Cleantalk/ApbctWP/Firewall/SFW.php |
Adds status-signed cookie helpers and integrates validation and issuance. |
cleantalk.php |
Uses the shared generator for AC/AF pass cookies. |
Review effort: Balanced
https://app.doboard.com/1/task/60124