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.

Sign up for GitHub

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

Jump to bottom

internal/appsec/waf: remove appsec rule size limitations (#1189) #1189

Merged
merged 4 commits into from Mar 1, 2022
Merged

internal/appsec/waf: remove appsec rule size limitations (#1189) #1189

merged 4 commits into from Mar 1, 2022

Commits on Mar 1, 2022

  1. internal/appsec/waf: do not limit the size of WAF rules 

    The size of the WAF rules were limited like regular WAF values, leading
to truncated WAF rules. This patch changes the limits to the max int
values when encoding the WAF rules. It is considered a simpler change
to achieve compared to the introduction of new "disabled limit" values
that would have introduced more advanced and complex conditions in the
encoder.
    @Julio-Guerra
    Julio-Guerra committed Mar 1, 2022
    Copy the full SHA
    200c4a4 View commit details
    Browse the repository at this point in the history

  2. internal/appsec/waf: fix undefined math.MaxInt

    @Julio-Guerra
    Julio-Guerra committed Mar 1, 2022
    Copy the full SHA
    42f73c7 View commit details
    Browse the repository at this point in the history

  3. Fix unused imports

    @Julio-Guerra
    Julio-Guerra committed Mar 1, 2022
    Copy the full SHA
    00bbfa5 View commit details
    Browse the repository at this point in the history

  4. Merge branch 'v1' into julio.guerra/unlimited-waf-rules

    @Julio-Guerra
    Julio-Guerra committed Mar 1, 2022
    Copy the full SHA
    28fe655 View commit details
    Browse the repository at this point in the history