Skip to content
This repository has been archived by the owner on Aug 3, 2023. It is now read-only.

1.19.6 #2159

Merged
merged 1 commit into from
Dec 16, 2021
Merged

1.19.6 #2159

merged 1 commit into from
Dec 16, 2021

Conversation

threepointone
Copy link
Contributor

  • Features

  • Fixes

    • Don't look for background updates unless Wrangler finished successfully - jyn514, pull/2150

      This works around a segfault due to OpenSSL's exit handlers not being thread-safe.

      See Segmentation fault after wrangler executes its command #1464 (comment) for an explanation and alternatives.

    • fix: incomplete binary with npm installation - 12f23eddde, pull/2149

      Closes wrangler binary is not complete when installing using npm #2148.
      This PR modifies binary-install.js (reference) to make sure the file stream is complete before the program finishes.
      I'm not a
      ... truncated

    • Get https websockets working - jyn514, pull/2153

      It turns out websocket upgrades with HTTP/2 require an HTTP extension,
      which Cloudflare doesn't currently support: https://datatracker.ietf.org/doc/html/rfc8441

      To avoid this, enable HTTP/1 for the remote client.

      This required an upd
      ... truncated

    • Get the audit CI job passing - jyn514, pull/2151

      Note that I didn't say "fix the vulnerabilities" - this just ignores the chrono and time vulnerabilities because they're both very hard to fix and not very common in practice.

      This uncovered a tokio vulnerability, which I've fixed by
      ... truncated

    • Proxy websocket connections when using authenticated (realish) preview - jyn514, pull/2135

      Previously, Wrangler would return a "101 Switching Protocols" response
      and then immediately close the TCP connection. This changes it to instead
      continue proxying the connection to the remote worker.

      This is simpler than `wrangler dev
      ... truncated

  • Maintenance

    • Update Rust toolchain to 1.57 - taylorlee, pull/2145

      I originally updated my toolchain to 1.56.1 on a local branch, since afaict rust-analyzer wasn't able to work with derived StructOpt proc-macros in 1.54. Since 1.57 was released today, I figured I'd update it all the way in this pr.

      foll
      ... truncated

@threepointone threepointone requested review from a team December 16, 2021 14:39
Copy link
Contributor

@jyn514 jyn514 left a comment

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

Looks fine to me - a bunch of the dependency versions got updated but that may be intentional?

Sign up for free to subscribe to this conversation on GitHub. Already have an account? Sign in.
Labels
None yet
Projects
None yet
Development

Successfully merging this pull request may close these issues.

wrangler binary is not complete when installing using npm
2 participants