Does not play well with koa-compose

App always sends 404 Not found when koa-bodyparser is wrapped inside koa-compose

{
    "koa": "^2.4.1",
    "koa-bodyparser": "^2.5.0",
    "koa-compose": "^4.0.0"
}

const Koa = require('koa')
const compose = require('koa-compose')
const bodyparser = require('koa-bodyparser')

const app = new Koa()

// app.use(bodyparser()) // works
app.use(compose([bodyparser()])) // send 404

app.use((ctx) => {
  ctx.body = 'Hello World!'
})

app.listen(3000)

[codinggirl]

When meet this, we should check other middleware’s calling of next().

If a middleware is a common function, we should call like:

return next()

But not:

next()

The solution is add keyword return and let the middleware returns the promise.

I struggle this for several hours, so write it down, may it helps.

Finally, see this article for more detail.

[fl0w]

This issue is invalid.

@yadwinderpaul You’re using an old version of koa-bodyparse which is not compliant with Koa 2 (and thus compose 4 and 3 which is what Koa is using currently).

@codinggirl It makes no sense to break the promise flow with synchronous middleware currently. And not calling next is a completely valid use case. Having a guard in this repo (even though one could assume next is always called after this specific middleware) can be very confusing.