Skip to content
This repository has been archived by the owner on Aug 29, 2018. It is now read-only.

Shouldn't it be authentication instead of the old auth there? #11

Closed
huafu opened this issue Mar 28, 2017 · 1 comment
Closed

Shouldn't it be authentication instead of the old auth there? #11

huafu opened this issue Mar 28, 2017 · 1 comment

Comments

@huafu
Copy link

huafu commented Mar 28, 2017

Following the latest feathers-authentication, I believe on this line https://github.com/feathersjs/feathers-authentication-local/blob/master/src/hooks/hash-password.js#L14 the config should be read from authentication and not auth.
@marshallswain
Copy link
Member

@huafu You are right. Good catch.

@marshallswain marshallswain mentioned this issue Mar 28, 2017
Merged
Sign up for free to subscribe to this conversation on GitHub. Already have an account? Sign in.
Assignees
No one assigned
Labels
None yet
Projects
None yet
Milestone
No milestone
Development

No branches or pull requests
2 participants
@marshallswain @huafu