koa-bodyparser seems dead now; @koa/bodyparser is the up-to-date version

[aseemk]

Hi there. Thanks for this library! We've been using it and it's helpful.

I recently ran into an issue unrelated to Apollo, but related to the Koa body-parsing middleware this library expects and requires.

We installed koa-bodyparser which this library's readme recommends. However, you can see that package is stuck on version 4.

If you go to the Koa bodyparser's GitHub repo, it actually specifies the package name as @koa/bodyparser — and that package is on version 5 now (and actually maintained).

I had to use a new option added to version 5 to fix a bug, and couldn't understand why there was no version 5 on npm.

Please consider upgrading not just the readme here but also the code and test dependencies on this package.

Thanks again and hope this helps others!

[aseemk]

Another issue related to this: going from v4 (koa-bodyparser) to v5, @koa/bodyparser now no longer populates ctx.request.body (even to {}) on GET requests:

https://github.com/koajs/bodyparser/blob/664cd7c413250d5e12eb5bb0fbf4e52d31ef24f5/src/body-parser.ts#L30

https://github.com/koajs/bodyparser/blob/664cd7c413250d5e12eb5bb0fbf4e52d31ef24f5/src/body-parser.ts#L76-L87

Makes sense since GET requests have no body, so should need no parsing.

But this Apollo Koa integration still indiscriminately expects a ctx.request.body to be set even if the method is GET (for loading the Apollo GraphiQL UI):

apollo-server-integration-koa/src/index.ts

Lines 72 to 82 in 71b7ba3

	return async (ctx) => {
	if (!ctx.request.body) {
	// The json koa-bodyparser *always* sets ctx.request.body to {} if it's unset (even
	// if the Content-Type doesn't match), so if it isn't set, you probably
	// forgot to set up koa-bodyparser.
	ctx.status = 500;
	ctx.body =
	'`ctx.request.body` is not set; this probably means you forgot to set up the ' +
	'`koa-bodyparser` middleware before the Apollo Server middleware.';
	return;
	}

I'm hackily setting ctx.request.body ??= {} myself in my own wrapper middleware if ctx.method === 'GET', and this works. But should this be fixed here in this library? Thank you!

[matthew-gordon]

Hi @aseemk, please refer to this previous comment: #60 (comment)

Ultimately I agree It is certainly a good idea to use packages that are actively maintained, if you feel strongly I definitely encourage a PR with the changes.