Skip to content

Make the boundary check less strict as per RFC2046 #1955

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 2 commits into from
Closed

Make the boundary check less strict as per RFC2046 #1955

wants to merge 2 commits into from

Conversation

victorhora
Copy link
Contributor

@victorhora victorhora commented Nov 14, 2018

This PR is based on #1821 but directed at the v3 branch and with a test case to validate the changes.

According to the RFC2046 page 22, indeed the parser seemed to be more strict than it should. I believe this is safe to be merged.

@zimmerle
Copy link
Contributor

Merged! thanks

@zimmerle zimmerle closed this Nov 21, 2018
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
3.x Related to ModSecurity version 3.x enhancement RIP - libmodsecurity
Projects
None yet
Development

Successfully merging this pull request may close these issues.

2 participants