themarshallproject / klaxon

Klaxon enables reporters and editors to monitor scores of sites on the web for newsworthy changes.
https://newsklaxon.org
MIT License
646 stars 199 forks source link

Move to using Rails `session` cookie for user ID source #639

Open rdmurphy opened 1 year ago

rdmurphy commented 1 year ago

It's possible Klaxon just predates this being a thing, but there's no reason for us to use cookie[:user_id] — this is what the session cookie is for.

rdmurphy commented 1 year ago

Actually now I see it's this way for persistence reasons. 😅 Still could be tidied up.