What does this MR do?
Gitlab::Identifier so that it:
- Has tests
- Caches output in an instance variable to reduce queries
- Uses only a single query to find a user by an SSH key, instead of 2
Why was this MR needed?
This code was untested and would execute more SQL queries than needed.