menems / koa-66 Goto Github PK

If routes like this are used

router.get('/appointments/validate', appointments.validate)
router.get('/appointments/:id', appointments.fetch)

the appointments.validate middleware gets an 'id' param with the value 'validate'.

Here is my code

app.use((ctx, next) => {
  console.info('1')
  next().then(result => {
    console.log('3')
    console.log(result) // expect hello
  })
})

app.use(apiRouter.routes())

const router = new Router()
const apiRouter = new Router()

router.get('/apps', (ctx) => {
  console.log('2')
  return Promise.resolve('hello')
})

apiRouter.mount('/api', router)
module.exports = apiRouter

but terminal shows

1
2
3
undefined

If you use 2 routes using the same plugin such as :

.delete('/:id', { acl: ['HELPDESK'] }, (ctx) => {}

and

.put('/user/validate', { acl: ['DIRECTION'] }, (ctx) => {}

Your second route will take { acl: ['HELPDESK'] } instead of { acl: ['DIRECTION'] } as plugin.

Instead of setting methods in the contructor put them in the prototype where all the other methods are.

very important feature

task list

• comments
• more Test (async missing)
• Docs
• Consolidate input test
• Add multiple middleware support
• 405 Method Not Allowed and 501 Not Implemented.

I'm pubilshing a v1.0.0 version , now that koa has 2.0.0 tag on github/npm
There is no change since the last version, and i think this project is enought stable to have a v1.

So don't panic if you see a v1 version this is a same version than v0.8.5. it 's just a semantic update and some devdependencies update

thanks

Hi, it would be great if koa-66 support route.all(...) like koa-router. Currently I hack around my code by defining a route for every method in response to all, but that's terribly inefficient.

I used this in my project a lot and I would like to move to koa-66.