Skip to content
This repository has been archived by the owner on Apr 22, 2023. It is now read-only.

Feature dos collision limits 2 #2542

Closed
wants to merge 2 commits into from
Closed

Feature dos collision limits 2 #2542

wants to merge 2 commits into from

Conversation

indutny
Copy link
Member

@indutny indutny commented Jan 16, 2012

nt

@indutny
http: do not accept headers if limit is exceeded
664eb8a 
* fix limiting when parser was reused
* fix maxHeadersCount = 0 is ignored ( credit to @koichik )
* add test-http-max-headers-count.js ( credit to @koichik )
@indutny
querystring: fix maxKeys = 0 is ignored
7eed288
@indutny indutny mentioned this pull request Jan 16, 2012
Closed
@indutny
Copy link
Member Author

indutny commented Jan 16, 2012

merged in rf1678bf and r23de339. /cc @koichik

@indutny indutny closed this Jan 16, 2012
Sign up for free to subscribe to this conversation on GitHub. Already have an account? Sign in.
Reviewers
No reviews
Assignees
No one assigned
Labels
http querystring
Projects
None yet
Milestone
No milestone
Development

Successfully merging this pull request may close these issues.
1 participant
@indutny