• 
      

    Fix some string type issues involving hosting service headers.

    Review Request #10824 — Created Jan. 13, 2020 and submitted — Latest diff uploaded

    Information

    Review Board
    release-4.0.x

    Reviewers

    The GitHub implementation was assuming use of byte strings for headers,
    as that's what urllib2 wants on Python 2. On Python 3, however, Unicode
    strings are used. This means we need to respect the native string type
    on both versions.

    This change updates the GitHub code to use native string types when
    dealing with headers, and updates the tests as well.

    It also fixes the type check in HostingServiceHTTPResponse, so that we
    catch these problems everywhere.

    Unit tests pass on Python 2.x and 3.x.

    Commits

    Files