Document CrowdSec AppSec request inspection for the reverse proxy - #889
Document CrowdSec AppSec request inspection for the reverse proxy#889lixmal wants to merge 2 commits into
Conversation
|
The latest updates on your projects. Learn more about Vercel for GitHub.
|
|
No actionable comments were generated in the recent review. 🎉 ℹ️ Recent review info⚙️ Run configurationConfiguration used: Organization UI Review profile: CHILL Plan: Pro Plus Run ID: 📒 Files selected for processing (4)
🚧 Files skipped from review as they are similar to previous changes (4)
📝 WalkthroughWalkthroughThe documentation adds CrowdSec AppSec (WAF) guidance for reverse-proxy access restrictions, access logs, self-hosted configuration, and migration setup, including inspection limits and enforce/observe behavior. ChangesCrowdSec AppSec integration
Estimated code review effort: 3 (Moderate) | ~20 minutes Possibly related PRs
Suggested reviewers: Poem
🚥 Pre-merge checks | ✅ 5✅ Passed checks (5 passed)
✨ Finishing Touches🧪 Generate unit tests (beta)
Warning There were issues while running some tools. Please review the errors and either fix the tool's configuration or disable the tool if it's a critical failure. 🔧 ESLint
src/pages/manage/reverse-proxy/access-logs.mdxOops! Something went wrong! :( ESLint: 9.39.2 TypeError: Converting circular structure to JSON src/pages/manage/reverse-proxy/authentication.mdxOops! Something went wrong! :( ESLint: 9.39.2 TypeError: Converting circular structure to JSON src/pages/selfhosted/maintenance/crowdsec.mdxOops! Something went wrong! :( ESLint: 9.39.2 TypeError: Converting circular structure to JSON
Thanks for using CodeRabbit! It's free for OSS, and your support helps us grow. If you like it, consider giving us a shout-out. Comment |
7f695a7 to
8eb9c2b
Compare
8eb9c2b to
ba1a479
Compare
ba1a479 to
c8747b6
Compare
c8747b6 to
d2c7e49
Compare
There was a problem hiding this comment.
Actionable comments posted: 5
🤖 Prompt for all review comments with AI agents
Verify each finding against current code. Fix only still-valid issues, skip the
rest with a brief reason, keep changes minimal, and validate.
Inline comments:
In `@src/pages/manage/reverse-proxy/access-logs.mdx`:
- Around line 73-75: Update the access-log reason documentation near the
`appsec_ban`, `appsec_captcha`, and `appsec_unavailable` rows to identify these
`appsec_*` reasons as HTTP-only. Adjust the surrounding service-availability
statement or split the table labeling so it no longer claims these reasons apply
to L4 services, while preserving the existing descriptions.
In `@src/pages/manage/reverse-proxy/authentication.mdx`:
- Around line 378-379: Update the ordered-list marker for the step following the
CrowdSec AppSec request inspection instructions so it uses 7 instead of 6,
preserving the existing step text and sequence.
- Around line 278-280: Update the CrowdSec setup link in
src/pages/manage/reverse-proxy/authentication.mdx:278-280 to use the decamelized
`#app-sec-waf-request-inspection` anchor. Update the corresponding links in
src/pages/selfhosted/maintenance/crowdsec.mdx:173-173 to
`#crowd-sec-ip-reputation` and
src/pages/selfhosted/migration/enable-reverse-proxy.mdx:411-411 to
`#app-sec-waf-request-inspection`.
In `@src/pages/selfhosted/maintenance/crowdsec.mdx`:
- Around line 95-96: Replace the exposed internal CrowdSec hostname with a
deployment-specific placeholder in src/pages/selfhosted/maintenance/crowdsec.mdx
lines 95-96 and lines 125-127, and in
src/pages/selfhosted/migration/enable-reverse-proxy.mdx lines 400-411. Update
the surrounding documentation to state that the placeholder must be replaced
with the deployment’s CrowdSec service hostname.
- Line 117: Close the currently open YAML code fence before the prose in the
CrowdSec maintenance documentation, ensuring the subsequent bash and text
examples render as separate examples. Preserve the explanatory prose and
existing content unchanged.
🪄 Autofix (Beta)
Fix all unresolved CodeRabbit comments on this PR:
- Push a commit to this branch (recommended)
- Create a new PR with the fixes
ℹ️ Review info
⚙️ Run configuration
Configuration used: Organization UI
Review profile: CHILL
Plan: Pro Plus
Run ID: 70bdebc4-1f11-48d3-88b6-5a27b13d8ba1
📒 Files selected for processing (4)
src/pages/manage/reverse-proxy/access-logs.mdxsrc/pages/manage/reverse-proxy/authentication.mdxsrc/pages/selfhosted/maintenance/crowdsec.mdxsrc/pages/selfhosted/migration/enable-reverse-proxy.mdx
Documents CrowdSec AppSec (WAF) request inspection for the reverse proxy: a per-service mode that mirrors each HTTP request to the Security Engine and blocks or logs what its rules flag.
appsec_*deny reasons and theappsec_body_bypassmetadata in the access-logs referenceNeed help on this PR? Tag
@codesmith-botwith what you need. Autofix is disabled.Summary by CodeRabbit