Agreed on the unnecessary ``self``s, it was a consequence of the fieldname evolving as I wrote the code.

There's also ``find_by(key: value)`` in Rails 4 which is more concise. We're using it in Bouncer here: https://github.com/alphagov/bouncer/blob/master/lib/bouncer/request_context.rb#L13