Skip to content

fix: Send null for uploaded files in multipart operations - #1754

Open
knd775 wants to merge 1 commit into
HoudiniGraphql:mainfrom
knd775:fix/multipart-operations-null
Open

knd775 wants to merge 1 commit into
HoudiniGraphql:mainfrom
knd775:fix/multipart-operations-null

Conversation

@knd775

@knd775 knd775 commented Sep 29, 2026 •

Copy link
Copy Markdown
Contributor

JSON.stringify serializes a File into {}, which violates the multipart upload spec and breaks requests to some strict servers (in our case, Hot Chocolate starting with version 16)

To help everyone out, please make sure your PR does the following:

  • Update the first line to point to the ticket that this PR fixes
  • Add a message that clearly describes the fix
  • If applicable, add a test that would fail without this fix
  • Make sure the unit and integration tests pass locally with pnpm run tests and cd integration && pnpm run tests
  • Includes a changeset if your fix affects the user with pnpm changeset

Provided bodies are patched too: defaultFetch and plugins that set fetchParams.body serialize each File as {}, so nulling only the fallback body wouldn't fix either.
@changeset-bot

changeset-bot Bot commented Sep 29, 2026

Copy link
Copy Markdown

🦋 Changeset detected

Latest commit: 08642bc

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

This PR includes changesets to release 2 packages
Name Type
houdini Patch
houdini-core 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

This branch has not been deployed

No deployments
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Labels

None yet

Projects

None yet

Development

Successfully merging this pull request may close these issues.

1 participant