-
Notifications
You must be signed in to change notification settings - Fork 25
New issue
Have a question about this project? Sign up for a free GitHub account to open an issue and contact its maintainers and the community.
By clicking “Sign up for GitHub”, you agree to our terms of service and privacy statement. We’ll occasionally send you account related emails.
Already on GitHub? Sign in to your account
Support for Hapi v17 #25
Conversation
* Updated dependencies. Fixed issue with server.connections used for route iteration before. * Partial fix for the tests. * Made logic around ACL queries asynchronous. Fixed part of tests. * Fixed remaining errors and tests. * Updated README
@toymachiner62 I've tested it with my use-case and no issue. |
- '5' | ||
- '4.2.0' | ||
- '4' | ||
- 8 |
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
Any reason we're skipping node 9?
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
I took it from hapijs, I guess they only officially support LTS
Available on npm version 4.0.0 |
Version 17.x only supports node v8.9.0 and over.
)var
tolet
/const
register
syntax for pluginsq
to use native promiseroles
toaclQuery
ifrole
is undefinedThx @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