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

Changing authenticateMiddleware to return the auth function. #127

Merged
merged 1 commit into from
Jul 4, 2017
Merged

Changing authenticateMiddleware to return the auth function. #127

merged 1 commit into from
Jul 4, 2017

Conversation

kingjerod
Copy link
Contributor

I'm not sure why I didn't have this issue before when I was testing out security, but when using the @Security decorator, it is causing a 404 on that route. Through a lot of trial and error I narrowed it down to missing this return statement. I've tested it in Koa but am not familiar enough with Express/Hapi to test it there.

Maybe this is some fluke on my end and this is not needed? I cannot get @Security to work otherwise.

@lukeautry lukeautry merged commit d03c94f into lukeautry:master Jul 4, 2017
@lukeautry
Copy link
Owner

Thanks @kingjerod! Will release this in the next update.

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.

2 participants