Skip to content

Fixed no double newline issue #25

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.

Sign up for GitHub

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

Open
wants to merge 7 commits into
base: master
Choose a base branch
from
Open

Fixed no double newline issue #25

Show file tree
Hide file tree
Changes from all commits
Commits
Show all changes
7 commits
Select commit Hold shift + click to select a range
1d703e8
fixed cleanup_pool
stop777 Mar 19, 2013
c07e8ca
Fixed process_func setting in AsyncClient.start
stop777 Mar 19, 2013
0bb00b1
Fixed exception, when misconfigured server returns more than one stat…
stop777 Mar 20, 2013
250148c
Merge branch 'master' of github.com:stop777/human_curl
stop777 Mar 20, 2013
5a44074
Revert "Fixed process_func setting in AsyncClient.start"
stop777 Mar 20, 2013
4a74397
Merge branch 'master' of https://github.com/Lispython/human_curl
stop777 Mar 20, 2013
0025f5d
Fixed no exception when no newline at end of headers when allow_redir…
stop777 Mar 20, 2013
File filter

Filter by extension

Filter by extension
Conversations
Failed to load comments.
Loading
Jump to
Jump to file
Failed to load files.
Loading
Diff view
Diff view
1 change: 1 addition & 0 deletions human_curl/async.py
View file
Open in desktop
Original file line number Diff line number Diff line change
Expand Up @@ -298,6 +298,7 @@ def cleanup_pool(self):
opener.close()

self._openers_pool.close()
self._openers_pool = None

def method(self, method, **kwargs):
"""Added request params to data_queue
Expand Down
14 changes: 12 additions & 2 deletions human_curl/core.py
View file
Open in desktop
Original file line number Diff line number Diff line change
Expand Up @@ -727,6 +727,12 @@ def json(self):
except ValueError:
return None

def _clean_raw_headers(self, raw_headers):
ret = raw_headers.strip()
ret = ret.replace("\r\nHTTP", "\r\n\r\nHTTP")
ret = ret.replace("\r\n\r\n\r\n", "\r\n\r\n")
return ret

def _parse_headers_raw(self):
"""Parse response headers and save as instance vars
"""
Expand Down Expand Up @@ -763,15 +769,19 @@ def parse_header_block(raw_block):
logger.warn(e)
continue
else:
block_headers.append((version, code, message))
if len(block_headers) > 0:
logger.warn("Status HTTP header already exitst %s, but found one more %s for url %s" %
(block_headers[0], (version, code, message), self.url))
else:
block_headers.append((version, code, message))
else:
# raise ValueError("Wrong header field")
pass
return block_headers

raw_headers = self._headers_output.getvalue()

headers_blocks = raw_headers.strip().split("\r\n\r\n")
headers_blocks = self._clean_raw_headers(raw_headers).split("\r\n\r\n")
for raw_block in headers_blocks:
block = parse_header_block(raw_block)

Expand Down