net-cpp

Merge lp://staging/~gary-wzl77/net-cpp/fix_1631846 into lp://staging/net-cpp

  1. fix_1631846
  2. Merge into trunk
Proposed by Gary.Wang
Status: Merged
Approved by: Thomas Voß
Approved revision: 55
Merged at revision: 52
Proposed branch: lp://staging/~gary-wzl77/net-cpp/fix_1631846
Merge into: lp://staging/net-cpp
Diff against target: 86 lines (+44/-2)
4 files modified
CMakeLists.txt (+1/-1)
debian/changelog (+8/-0)
src/core/net/http/impl/curl/request.h (+1/-1)
tests/http_client_test.cpp (+34/-0)
To merge this branch: bzr merge lp://staging/~gary-wzl77/net-cpp/fix_1631846
Reviewer Review Type Date Requested Status
Thomas Voß (community) Approve
James Henstridge Approve
Review via email: mp+308017@code.staging.launchpad.net

Commit message

Fixes http header parsing in net response.

Description of the change

Fixes http header parsing in net response.

The first index of sub_match is full match.The actual index for sub_match[key, value] starts from 1.

To post a comment you must log in.
lp://staging/~gary-wzl77/net-cpp/fix_1631846 updated
53. By Gary.Wang

enable the unit test(get_request_against_app_store_succeeds) to verify if headers are parsed properly.

Revision history for this message
James Henstridge (jamesh) wrote :

I'm not really comfortable with the test changes. See inline comments.

review: Needs Fixing
lp://staging/~gary-wzl77/net-cpp/fix_1631846 updated
54. By Gary.Wang

add one test case to verify header parsing properly.

55. By Gary.Wang

simplify the test case to only verify http header and avoid the noise.

Revision history for this message
James Henstridge (jamesh) wrote :

Looks good.

review: Approve
Revision history for this message
Thomas Voß (thomas-voss) wrote :

LGTM.

review: Approve
lp://staging/~gary-wzl77/net-cpp/fix_1631846 updated
56. By Gary.Wang

add a log to check why it failed to compile against zesty.
suspects the content-length doesn't match on zesty.

57. By Gary.Wang

The value of "Content-Length" are different on xenial and zesty,
so we only check key and ignore value for it. That makes http_client_tests passed.

Preview Diff

[H/L] Next/Prev Comment, [J/K] Next/Prev File, [N/P] Next/Prev Hunk
The diff is not available at this time. You can reload the page or download it.

Subscribers

People subscribed via source and target branches

to all changes: