Alexandroppolus

На мой взгляд, небольшое нарушение в том, что модуль самостоятельно резолвит свою зависимость от UserRepo, а не получает её через DI. От Password, наверно, тоже, но это вроде как утилита и можно не заморачиваться. А вот UserRepo я бы заинжектил.