Skip to content

Commit 32c674d

Browse files
authored
Merge pull request #3 from fabrix-app/v1.1
[chore] better reporting for next release
2 parents 19bdd9f + 2bb6b49 commit 32c674d

File tree

2 files changed

+6
-8
lines changed

2 files changed

+6
-8
lines changed

lib/utils.ts

Lines changed: 3 additions & 5 deletions
Original file line numberDiff line numberDiff line change
@@ -25,20 +25,18 @@ export const Utils = {
2525
route.config.pre = route.config.pre || (route.config.pre = [ ])
2626

2727
path = Utils.getPathFromRoute(app, path, route)
28-
// route.handler = Utils.getHandlerFromString(app, rawRoute.handler)
2928
Utils.getHandlerFromString(app, route)
3029

31-
3230
route.config.pre = route.config.pre
3331
.map(pre => this.getHandlerFromPrerequisite(app, pre))
3432
.filter(handler => !!handler)
3533

36-
const routeHandlers = Object.keys(route)
34+
const routeHandlers = Object.keys(route).filter(value => -1 !== Utils.methods.indexOf(value))
3735

3836
if (!routeHandlers.some(v => Utils.methods.indexOf(v) >= 0)) {
39-
app.log.error('spool-router: route handler [', routeHandlers.join(', '), ']',
37+
app.log.error('spool-router: route ', path, ' handler [', routeHandlers.join(', '), ']',
4038
'does not correspond to any defined Controller handler')
41-
return
39+
return {}
4240
}
4341
return { path, route }
4442
},

package-lock.json

Lines changed: 3 additions & 3 deletions
Some generated files are not rendered by default. Learn more about customizing how changed files appear on GitHub.

0 commit comments

Comments
 (0)