Skip to content

Navigation Menu

Sign in
Appearance settings

Search code, repositories, users, issues, pull requests...

Provide feedback

We read every piece of feedback, and take your input very seriously.

Saved searches

Use saved searches to filter your results more quickly

Appearance settings

Conversation

antonio-ivanovski
Copy link
Contributor

@antonio-ivanovski antonio-ivanovski commented Mar 28, 2025

This PR adds logic for handling logic where the content-type and content-length are incorrectly set. This is not invalid by spec, just it doesn't make sense. However there are servers and middlewares that will take the opportunity to mess-up the perfectly valid empty-body response.

Copy link

vercel bot commented Mar 28, 2025

@antonio-ivanovski is attempting to deploy a commit to the ts-rest Team on Vercel.

A member of the Team first needs to authorize it.

Copy link

changeset-bot bot commented Mar 28, 2025

🦋 Changeset detected

Latest commit: a846740

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

This PR includes changesets to release 13 packages
Name Type
@ts-rest/core Patch
@ts-rest/example-contracts Patch
@ts-rest/non-strict-mode-test Patch
@ts-rest/express Patch
@ts-rest/fastify Patch
@ts-rest/nest Patch
@ts-rest/next Patch
@ts-rest/open-api Patch
@ts-rest/react-query-v5 Patch
@ts-rest/react-query Patch
@ts-rest/serverless Patch
@ts-rest/solid-query Patch
@ts-rest/vue-query 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

Copy link

nx-cloud bot commented Mar 28, 2025

View your CI Pipeline Execution ↗ for commit a846740.

Command Status Duration Result
nx affected -t lint,test,build ✅ Succeeded 3m 5s View ↗
nx run-many --target=build --projects=ts-rest* ✅ Succeeded 54s View ↗

☁️ Nx Cloud last updated this comment at 2025-03-28 12:13:51 UTC

Copy link

@antonio-ivanovski antonio-ivanovski changed the title handle zero content-length header Graceful handling of empty body content but invalid content-type/content-length response headers Mar 28, 2025
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

Morty Proxy This is a proxified and sanitized view of the page, visit original site.