feathersjs-ecosystem / authentication-local

[MOVED] Local authentication plugin for feathers-authentication
https://github.com/feathersjs/feathers
MIT License
26 stars 15 forks source link

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

Closed huafu closed 7 years ago

huafu commented 7 years ago

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 commented 7 years ago

@huafu You are right. Good catch.