kuzzleio / kuzzle

Open-source Back-end, self-hostable & ready to use - Real-time, storage, advanced search - Web, Apps, Mobile, IoT -
https://kuzzle.io
Apache License 2.0
1.44k stars 124 forks source link

Login defaults strategy to "local" instead of throwing an error #754

Closed scottinet closed 7 years ago

scottinet commented 7 years ago

Because authentication strategies are plugins, and because there is no default authentication strategy implemented in Kuzzle, I think we should not default the login strategy to local. Instead, we should make this argument required, and throw a BadRequestError if no strategy is provided (see AuthController:login)

xbill82 commented 7 years ago

Will be solved along with #831