Skip to content
New issue

Have a question about this project? Sign up for a free GitHub account to open an issue and contact its maintainers and the community.

By clicking “Sign up for GitHub”, you agree to our terms of service and privacy statement. We’ll occasionally send you account related emails.

Already on GitHub? Sign in to your account

[APPSEC-10967] add defaults scanners to the WAF ruleset #3166

Merged
merged 1 commit into from
Sep 28, 2023

Conversation

GustavoCaso
Copy link
Member

What does this PR do?

Add the default scanners to the WAF. The scanners are used to tag extracted schema information.

Motivation:

Additional Notes:

How to test the change?

For Datadog employees:

  • If this PR touches code that signs or publishes builds or packages, or handles
    credentials of any kind, I've requested a review from @DataDog/security-design-and-guidance.
  • This PR doesn't touch any of that.

Unsure? Have a question? Request a review!

@GustavoCaso GustavoCaso requested review from a team as code owners September 28, 2023 08:52
@github-actions github-actions bot added the appsec Application Security monitoring product label Sep 28, 2023
@GustavoCaso GustavoCaso changed the title add defaults scanners to the WAF ruleset [APPSEC-10967] add defaults scanners to the WAF ruleset Sep 28, 2023
@GustavoCaso GustavoCaso force-pushed the asm-add-default-processor-scanners branch from ee60596 to 57ae107 Compare September 28, 2023 09:03
Copy link
Member

@lloeki lloeki left a comment

Choose a reason for hiding this comment

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

LGTM

@GustavoCaso GustavoCaso merged commit 92ee7db into master Sep 28, 2023
181 checks passed
@GustavoCaso GustavoCaso deleted the asm-add-default-processor-scanners branch September 28, 2023 15:55
@github-actions github-actions bot added this to the 1.15.0 milestone Sep 28, 2023
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
appsec Application Security monitoring product
Projects
None yet
Development

Successfully merging this pull request may close these issues.

2 participants