Skip to content

http, separate response parsing from response action - #13134

Closed
icing wants to merge 6 commits into
curl:masterfrom
icing:http-resp-handling
Closed

icing wants to merge 6 commits into
curl:masterfrom
icing:http-resp-handling

Conversation

@icing

@icing icing commented Mar 15, 2024 •

Copy link
Copy Markdown
Contributor
  • move code that triggers on end-of-response into separate function from parsing
  • simplify some headp/headerlen usage
  • add httpversion to SingleRequest to indicate the version of the current response

@github-actions github-actions Bot added the tests label Mar 15, 2024
@icing
icing force-pushed the http-resp-handling branch 2 times, most recently from 41fa5f3 to 7773b8f Compare March 18, 2024 13:35
@bagder

bagder commented Mar 19, 2024

Copy link
Copy Markdown
Member

@icing if you could just take a look at the merge conflict...

icing added 6 commits March 19, 2024 09:08
- remove flags and timestamp from SingleRequest
- add 'exp100' client reader when announcing expectation
- new send flag KEEP_SEND_TIMED to trigger sending on
  timer events
- move code that triggers on end-of-response into separate
  function from parsing
- simplify some headp/headerlen usage
- add `httpversion` to SingleRequest to indicate the version
  of the current response
@icing
icing force-pushed the http-resp-handling branch from 7773b8f to 545c789 Compare March 19, 2024 08:27
@icing

icing commented Mar 20, 2024

Copy link
Copy Markdown
Contributor Author

@icing if you could just take a look at the merge conflict...

All resolved.

@bagder bagder closed this in 0f08b43 Mar 21, 2024
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Labels

Development

Successfully merging this pull request may close these issues.

2 participants