Support for Hapi v17 - Githubissues

NicolasRitouet commented 6 years ago

upgrade nodejs version on travis CI (Version 17.x only supports node v8.9.0 and over.)
move from var to let/ const
move to new register syntax for plugins
remove q to use native promise
update documentation with new plugin syntax
upgrade to major version 4
forwards roles to aclQuery if role is undefined

Thx @maxaus for his contribution.

I recommend having more people testing it. just modify your package.json from "hapi-authorization": "^3.0.2", to "hapi-authorization": "mycshq/hapi-authorization#hapi-17", and run npm i

coveralls commented 6 years ago

Coverage increased (+4.6%) to 100.0% when pulling c33534ae39b0aa65b10bd12fba1fe6e31b6d36f9 on mycsHQ:hapi-17 into 9d55ca389e43db7ae6a9b56abe67a3fb658ff188 on toymachiner62:master.

NicolasRitouet commented 6 years ago

@toymachiner62 I've tested it with my use-case and no issue. Anyone else has been able to test it? @maxaus @toddhickerson @heartstchr

toymachiner62 commented 6 years ago

Available on npm version 4.0.0

toymachiner62 / hapi-authorization

Support for Hapi v17 #25