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.

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

Jump to bottom

Fix for 'Failed to parse headers' warning #1439

Merged
merged 9 commits into from Sep 19, 2018
Merged

Fix for 'Failed to parse headers' warning #1439

merged 9 commits into from Sep 19, 2018

Commits on Sep 12, 2018

  1. Add test for spurious header warnings; urllib3#1438

    @timb07
    timb07 committed Sep 12, 2018
    Copy the full SHA
    0796637 View commit details
    Browse the repository at this point in the history

  2. Simplify assert_header_parsing; fixes urllib3#1438

    @timb07
    timb07 committed Sep 12, 2018
    Copy the full SHA
    7e78f79 View commit details
    Browse the repository at this point in the history

  3. Revert "Simplify assert_header_parsing; fixes urllib3#1438" 

    This reverts commit 7e78f79.
    @timb07
    timb07 committed Sep 12, 2018
    Copy the full SHA
    5472700 View commit details
    Browse the repository at this point in the history

Commits on Sep 13, 2018

  1. Fix header test cases to remove unintended header separator 

    b'\r\n'.join(headers) means the headers passed into _test_broken_header_parsing don't need to be terminated with b'\r\n'; however the final header needs b'\r\n\r\n' appended
    @timb07
    timb07 committed Sep 13, 2018
    Copy the full SHA
    129a895 View commit details
    Browse the repository at this point in the history

  2. Improve test to check for unparsed_data value

    @timb07
    timb07 committed Sep 13, 2018
    Copy the full SHA
    7b13457 View commit details
    Browse the repository at this point in the history

  3. Rename header parsing test and fix so it passes; fixes urllib3#1438

    @timb07
    timb07 committed Sep 13, 2018
    Copy the full SHA
    5c0f06c View commit details
    Browse the repository at this point in the history

  4. Updated changelog

    @timb07
    timb07 committed Sep 13, 2018
    Copy the full SHA
    7649111 View commit details
    Browse the repository at this point in the history

Commits on Sep 17, 2018

  1. Merge branch 'master' into bug1438

    @sethmlarson
    sethmlarson committed Sep 17, 2018
    Copy the full SHA
    9fff9af View commit details
    Browse the repository at this point in the history

Commits on Sep 18, 2018

  1. Remove platform-specific marker

    @timb07
    timb07 committed Sep 18, 2018
    Copy the full SHA
    f22b64b View commit details
    Browse the repository at this point in the history