snapframework / snap

Top-level package for the official Snap Framework libraries, includes the snaplets API as well as infrastructure for sessions, auth, and templates.
http://snapframework.com/
BSD 3-Clause "New" or "Revised" License
455 stars 68 forks source link

RFC: auth snaplet user lookup by email #193

Closed imalsogreg closed 7 years ago

imalsogreg commented 7 years ago

I needed this when implementing email-based password reset. Is there interest in getting it cleaned up for upstreaming? I can update snaplet-postgresql-simple for this change. Are there any other commonly used auth snaplet backends I should update?

mightybyte commented 7 years ago

Looks good to me. Is this ready to merge?

imalsogreg commented 7 years ago

@mightybyte Yep. I've now tested it with the builtin cookie backend, postgresql-simple, sqlite, and redis.