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

Patch 1 #884

Closed
wants to merge 153 commits into from
Closed

Patch 1 #884

wants to merge 153 commits into from

Conversation

viotti
Copy link
Contributor

@viotti viotti commented May 25, 2016

What these changes does?

This is just a small documentation correction.

How to test your changes?

No functional changes.

Related issue number

None.

Checklist

Does not apply.

luhn and others added 30 commits January 19, 2016 08:14
Conflicts:
	requirements-dev.txt
	tests/test_helpers.py
	tests/test_signals.py
Clear deprecated `aiohttp.request` in the examples.
Fix typo (HTTPMEthodNowAllowed)
Daniel Nelson and others added 23 commits May 4, 2016 15:48
Fix leak of connection slot during connection error
Fix the code to use event loop signaling API
Implement a gunicorn worker that uses uvloop
This updates the MultipartReader to ignore the preamble of a multipart
message.  To quote the RFC:

> There appears to be room for additional information prior to the first
> boundary delimiter line and following the final boundary delimiter line.
> These areas should generally be left blank, and implementations must
> ignore anything that appears before the first boundary delimiter line or
> after the last one.

To do this, the MultipartReader now acts slightly differently at the
beginning of the file.  Instead of looking for a boundary and excepting
if it doesn't find one, it will skip over the initial data looking for
the first boundary and start reading from there.

If it doesn't find any boundary, it will except similar to how it did
before.

Fixes #880
Don't send body in response to HEAD request
Skip the preamble in MultipartReader.
Implement BasicAuth decode classmethod.
@coveralls
Copy link

coveralls commented May 25, 2016

Coverage Status

Coverage decreased (-0.2%) to 98.212% when pulling b0f644c on viotti:patch-1 into bc10d71 on KeepSafe:0.21.

@viotti
Copy link
Contributor Author

viotti commented May 25, 2016

Please disregard this. I forked from master and created the pull request to branch 0.21. Deeply sorry.

The actual commit is a single line change on the documentation: b0f644c

@viotti viotti closed this May 25, 2016
@viotti
Copy link
Contributor Author

viotti commented May 25, 2016

This is the correct pull request: #885. Sorry for the inconvenience.

@lock
Copy link

lock bot commented Oct 29, 2019

This thread has been automatically locked since there has not been
any recent activity after it was closed. Please open a new issue for
related bugs.

If you feel like there's important points made in this discussion,
please include those exceprts into that new issue.

@lock lock bot added the outdated label Oct 29, 2019
@lock lock bot locked as resolved and limited conversation to collaborators Oct 29, 2019
Sign up for free to subscribe to this conversation on GitHub. Already have an account? Sign in.
Labels
Projects
None yet
Development

Successfully merging this pull request may close these issues.

None yet