Если ты контролируешь код контроллеров, то в общем ничего страшного не будет, если отдать туда entity и там уже мапить.
Если не контролируешь, то лучше отдавать полную дто, а дальше пусть делают, что хотят
Мне просто просто архитектурно не нравится то, что у нас сервис зависит от контроллера. Сущности зависят от таблиц в базе. DAO от сущностей. Сервис от dao. А теперь вдруг бац, и сервис зависит от вышестоящего контроллера. Из-за того, что мы внедряем в него маппер