Skip to content
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

upgrade to hapi latest, including fixing audit warnings #143

Open
wants to merge 1 commit into
base: master
Choose a base branch
from

Conversation

Icehunter
Copy link

fixed audit warnings and fixed issues around having to have both @hapi/X and X installed for this to work.

@O-Mutt
Copy link

O-Mutt commented Sep 30, 2020

Any chance we could get 👀 on this. looks like this will solve #142 and #141

@MFogleman
Copy link

Also wanted to request this. I ran into the same issues and hopped on to try to address this, only to find this great PR already open

@O-Mutt
Copy link

O-Mutt commented Nov 10, 2020

bump @z0mt3c

@steevhise
Copy link

bump. @z0mt3c are you not maintaining this anymore?

@zohairachmed
Copy link

Anyone found a solution to this? I'm running into similar issues where latest hapi doesn't work with old inert and vision. This library still uses old versions somehow. Should we fork it and apply the fix manually? Really could use the fixed version right about now :)

@O-Mutt
Copy link

O-Mutt commented Apr 10, 2023

Anyone found a solution to this? I'm running into similar issues where latest hapi doesn't work with old inert and vision. This library still uses old versions somehow. Should we fork it and apply the fix manually? Really could use the fixed version right about now :)

Yes, the fix has been PRd already ☝️

Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
None yet
Projects
None yet
Development

Successfully merging this pull request may close these issues.

5 participants