Skip to content

Only run completion checks for request bodies, if we have #2094

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

Closed
wants to merge 1 commit into from
Closed

Only run completion checks for request bodies, if we have #2094

wants to merge 1 commit into from

Conversation

rainerjung
Copy link

actually seen all of the body. In case ProcessPartial is configured
and our input filter only processed part of the request, checking
for a complete request body makes no sense.

See issue #2093.

actually seen all of the body. In case ProcessPartial is configured
and our input filter only processed part of the request, checking
for a complete request body makes no sense.

See issue #2093.
@victorhora victorhora requested a review from zimmerle May 16, 2019 19:00
@victorhora victorhora added this to the v2.9.4 milestone May 16, 2019
@victorhora victorhora added 2.x Related to ModSecurity version 2.x enhancement labels May 16, 2019
@zimmerle
Copy link
Contributor

There is a discussion on issue #2093 about this pull request.

@rainerjung rainerjung closed this by deleting the head repository Oct 1, 2024
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
2.x Related to ModSecurity version 2.x enhancement
Projects
None yet
Development

Successfully merging this pull request may close these issues.

3 participants