Skip to content

Use the parsed request body rather than ctx.body - #132

Merged
enisdenjo merged 1 commit into
graphql:mainfrom
jmurzy:patch-1
Nov 6, 2024
Merged

Use the parsed request body rather than ctx.body#132
enisdenjo merged 1 commit into
graphql:mainfrom
jmurzy:patch-1

Conversation

@jmurzy

@jmurzy jmurzy commented Nov 4, 2024

Copy link
Copy Markdown
Contributor

The Koa adapter is incorrectly using the response body (ctx.body) instead of the request body (ctx.request.body).

@linux-foundation-easycla

linux-foundation-easycla Bot commented Nov 4, 2024

Copy link
Copy Markdown

CLA Signed

The committers listed above are authorized under a signed CLA.

  • ✅ login: jmurzy / name: Jake Murzy (cf9be7f)

@enisdenjo

Copy link
Copy Markdown
Member

Thanks for this PR! Can you please rebase and fix the failing lint+type checks?

@jmurzy

jmurzy commented Nov 6, 2024

Copy link
Copy Markdown
Contributor Author

@enisdenjo it should be fixed now. Thanks!

@enisdenjo
enisdenjo merged commit 9f8b1f1 into graphql:main Nov 6, 2024
@enisdenjo

Copy link
Copy Markdown
Member

Thank you @jmurzy, great work!

enisdenjo pushed a commit that referenced this pull request Nov 7, 2024
## [1.22.2](v1.22.1...v1.22.2) (2024-11-07)

### Bug Fixes

* **use/koa:** Use the parsed request body rather than ctx.body ([#132](#132)) ([9f8b1f1](9f8b1f1))
@enisdenjo

Copy link
Copy Markdown
Member

🎉 This PR is included in version 1.22.2 🎉

The release is available on:

Your semantic-release bot 📦🚀

@enisdenjo enisdenjo added the released Has been released and published label Nov 7, 2024
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Labels

released Has been released and published

Projects

None yet

Development

Successfully merging this pull request may close these issues.

2 participants