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

1.1.0 #243

Closed
wants to merge 1 commit into from
Closed

1.1.0 #243

wants to merge 1 commit into from

Conversation

ronag
Copy link
Member

@ronag ronag commented Jul 5, 2020 •
edited
Loading

  • Uses Node native HTTP parser. Improves performance and security. Removes dependency on http-parser-js.
  • Added headersTimeout option.
  • Fixed an assertion when using client.pipeline and the server does not consume the entire request body.
  • Replace client.full with client.busy property.
  • Improved support for legacy streams.
  • Added support for IPC (e.g. unix sockets) through socketPath option.
  • Refactoring and simplification.
  • Throw not supported error for CONNECT method.

@ronag ronag requested a review from mcollina July 5, 2020 17:00
@ronag
Copy link
Member Author

ronag commented Jul 5, 2020 •
edited
Loading

Also waiting for decision on #241 before release or some other resolution to #240.

@ronag
Copy link
Member Author

ronag commented Jul 5, 2020 •
edited
Loading

Replace client.full with client.busy property.

Kind of makes this semver major. WDYT? Alternatively we could just add full back (undocumented).

@ronag
Copy link
Member Author

ronag commented Jul 5, 2020

The switch to Node native parser has been a bit risky and I have found some misc bugs post the initial PR. But it feels pretty good now.

@mcollina
Copy link
Member

mcollina commented Jul 5, 2020

Ok for me!

@ronag
Copy link
Member Author

ronag commented Jul 6, 2020

published

@ronag ronag closed this Jul 6, 2020
@ronag ronag deleted the proposal-1.1 branch April 25, 2021 22:20
@snyk-bot snyk-bot mentioned this pull request Sep 15, 2021
Open
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Reviewers

@mcollina mcollina Awaiting requested review from mcollina

Assignees
No one assigned
Labels
None yet
Projects
None yet
Milestone
No milestone
Development

Successfully merging this pull request may close these issues.
2 participants
@ronag @mcollina