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.

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

[breaking] request creation cleanup #6681

Merged
merged 1 commit into from
Sep 9, 2022

Conversation

benmccann
Copy link
Member

A few things changed here:

  • allow HTTP/2 streaming. i.e. undefined content-length when using HTTP/2
  • I have no idea wtf err.reason was. Changed it to err.message
  • made getRequest take a single arg so that we can easily add a new option to it like bodySizeLimit (Limit size of post requests #6542) if we want the adapter to handle that option

@changeset-bot
Copy link

changeset-bot bot commented Sep 9, 2022

🦋 Changeset detected

Latest commit: c54ff93

The changes in this PR will be included in the next version bump.

This PR includes changesets to release 4 packages
Name Type
@sveltejs/adapter-node Patch
@sveltejs/adapter-vercel Patch
@sveltejs/kit Patch
@sveltejs/adapter-auto Patch

Not sure what this means? Click here to learn what changesets are.

Click here if you're a maintainer who wants to add another changeset to this PR

@Rich-Harris Rich-Harris merged commit f1e762b into sveltejs:master Sep 9, 2022
@github-actions github-actions bot mentioned this pull request Sep 9, 2022
@benmccann benmccann deleted the create-request branch September 9, 2022 05:02
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Projects
None yet
Development

Successfully merging this pull request may close these issues.

2 participants