Open zrisher opened 7 years ago
This PR adds a bit more detail to the readme around two major stumbling blocks for new users:
LocalStrategy
body-parser
req.body
passport-local
I haven't seen anyone complaining about the former, but it was a bit confusing for me and easy enough to fix as part of this PR.
There's plenty of examples of people having issues with the latter:
Coverage remained the same at 97.619% when pulling 3282b886b708a5f1faeac012f05737796656ccdc on zrisher:master into 3b9980f6467fd1203ecf8844fb017b5db2941606 on jaredhanson:master.
👍 this is so needed, quick glance would be so helpful with this information!
bump @jaredhanson
Merged in https://github.com/passport-next/passport-local
This PR adds a bit more detail to the readme around two major stumbling blocks for new users:
LocalStrategy
in the first code example comes frombody-parser
is required for fields to be available inreq.body
, wherepassport-local
expects them.I haven't seen anyone complaining about the former, but it was a bit confusing for me and easy enough to fix as part of this PR.
There's plenty of examples of people having issues with the latter: