hapijs / glue

Server composer for hapi.js
Other
245 stars 62 forks source link

options: {showAuth: true} throws error #86

Closed geekhunger closed 7 years ago

geekhunger commented 7 years ago

Is this me or is it a bug? As soon as I add showAuth to the options I get an error: "showAuth" is not allowed

Here's my manifest:

const manifest = { server: {}, connections: [ {host: "localhost", port: 8000} ], registrations: [ {plugin: "hapi-auth-basic"}, {plugin: "blipp", options: {showAuth: true}} ] }

Now the interesting part. If I remove showAuth from options and then go to the module _/nodemodules/blipp/lib/index.js and edit line 11 to be showAuth: Joi.boolean().default(true) // instead of false then it works!

How can this be?

WesTyler commented 7 years ago

That sounds like an issue with blipp, not glue. That error message is in fact a Joi error message, and you were able to fix the issue by modifying the blipp src.

geekhunger commented 7 years ago

oh, sorry! I didn't noticed that I posted this not in the blipp repo. (had both opened side by side)

lock[bot] commented 4 years ago

This thread has been automatically locked due to inactivity. Please open a new issue for related bugs or questions following the new issue template instructions.