Skip to content
This repository was archived by the owner on Apr 8, 2019. It is now read-only.

Assignment 4 #129

Closed
wants to merge 2 commits into from
Closed

Assignment 4 #129

wants to merge 2 commits into from

Conversation

idanwe
Copy link
Contributor

@idanwe idanwe commented Apr 28, 2015


return next();
});

Copy link
Contributor

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

no need for newline here

@FennNaten
Copy link
Contributor

Hi,
I can see that you didn't updated the package.json to add hapi-auth-basic module. Maybe you forgot to install it with --save flag.

handler: function (request, reply) {

var username = request.auth.credentials.username;
return reply(internals.respone(username));
Copy link
Contributor

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

little typo here, you typed 'respone' instead of 'response'

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

Successfully merging this pull request may close these issues.

Basic authentication
4 participants