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

Fix an issue causing a crash when the Content-Lenght was negative #457

Merged
merged 3 commits into from
Sep 26, 2020

Conversation

Vkt0r
Copy link
Member

@Vkt0r Vkt0r commented Jul 10, 2020

This PR can be focused on the following:

  • Fix an issue causing a crash when the Content-Lenght was negative
  • Add a unit tests for the new exception.
  • Fix a Swiftlint warning regarding the colon.
  • Fix a spelling error.

Fix #442.

* Add an unit tests for the new exception
* Fix an Swiftlint warning regarding the colon
* Fix a spelling error
@Vkt0r Vkt0r marked this pull request as ready for review September 23, 2020 15:46
@Vkt0r Vkt0r requested a review from pvzig September 23, 2020 15:46
@Vkt0r Vkt0r merged commit 73b7531 into httpswift:stable Sep 26, 2020
@Vkt0r Vkt0r deleted the content-length branch September 26, 2020 17:31
tomieq pushed a commit to tomieq/swifterfork that referenced this pull request Apr 1, 2021
Fix an issue causing a crash when the Content-Lenght was negative
@Vkt0r Vkt0r restored the content-length branch May 7, 2021 18:57
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
None yet
Projects
None yet
Development

Successfully merging this pull request may close these issues.

Request with "Content-Length: -1" fatalErrors
1 participant