search

igrigorik / em-http-request

Asynchronous HTTP Client (EventMachine + Ruby)
1.22k stars 220 forks source link

Add support for 'Expect: 100-continue' header #323

Closed CodeMonkeySteve closed 6 years ago

CodeMonkeySteve commented 6 years ago

This is how I fixed #321, it could probably use some refactoring

CodeMonkeySteve commented 6 years ago

After some research, I think this can be done in a non-hacky way by just looking for a 100 result code from the parser, then just resetting it and continuing. Let me write some tests (and rebase) and I'll get back to you.

CodeMonkeySteve commented 6 years ago

Made changes to handle 100-continue in a non-hacky way, and rebased on iostream branch.

igrigorik commented 6 years ago

@CodeMonkeySteve doh, looks like we need to rebase this once over. :)