It is easy to mix up `find_by_activation_token` and `load_from_activation_token`
joshbuker opened this issue · 0 comments
joshbuker commented
While writing tests for v1, I accidentally used find_by
instead of load_from
, which had me scratching my head as to why the test was finding the record even when set to nil. This is an easy mistake to make, and a potential security vulnerability - need to find some way to refactor this that makes it difficult or impossible to mistake.