Skip to content

add crowdsecurity/base-config to CRS collections for proper parsing of non URL-encoded bodies#1745

Merged
blotus merged 3 commits intomasterfrom
crs-rules-body-parsing
Apr 1, 2026
Merged

add crowdsecurity/base-config to CRS collections for proper parsing of non URL-encoded bodies#1745
blotus merged 3 commits intomasterfrom
crs-rules-body-parsing

Conversation

@blotus
Copy link
Copy Markdown
Member

@blotus blotus commented Mar 30, 2026

Copilot AI review requested due to automatic review settings March 30, 2026 20:18
Copy link
Copy Markdown
Contributor

Copilot AI left a comment

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

Pull request overview

Adds the crowdsecurity/base-config appsec rule to the CRS collections/configs so request bodies are parsed with the appropriate processor for non-application/x-www-form-urlencoded content types.

Changes:

  • Include crowdsecurity/base-config in CRS collections’ appsec-rules lists.
  • Load crowdsecurity/base-config in both out-of-band (crs) and in-band (crs-inband) appsec configs before CRS rules.

Reviewed changes

Copilot reviewed 4 out of 4 changed files in this pull request and generated 2 comments.

File Description
collections/crowdsecurity/appsec-crs.yaml Ensures base-config is installed alongside CRS when enabling the non-blocking CRS collection.
collections/crowdsecurity/appsec-crs-inband.yaml Ensures base-config is installed alongside CRS when enabling the blocking CRS collection.
appsec-configs/crowdsecurity/crs.yaml Loads base-config in out-of-band rules to set request body processors before CRS evaluation.
appsec-configs/crowdsecurity/crs-inband.yaml Loads base-config in in-band rules to set request body processors before CRS evaluation.

💡 Add Copilot custom instructions for smarter, more guided reviews. Learn how to get started.

Comment thread collections/crowdsecurity/appsec-crs.yaml Outdated
Comment thread collections/crowdsecurity/appsec-crs-inband.yaml Outdated
blotus and others added 2 commits March 30, 2026 22:23
Co-authored-by: Copilot <175728472+Copilot@users.noreply.github.com>
Co-authored-by: Copilot <175728472+Copilot@users.noreply.github.com>
@blotus blotus merged commit 41864fb into master Apr 1, 2026
11 checks passed
@blotus blotus deleted the crs-rules-body-parsing branch April 1, 2026 07:23
blotus added a commit that referenced this pull request Apr 16, 2026
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Labels

None yet

Projects

None yet

Development

Successfully merging this pull request may close these issues.

CrowdSec Appsec doesn’t process JSON/XML for CRS in out-of-band mode

3 participants