А где вы видели то, что у вас во втором примере?
Так не имеет смысла писать, это не особо "функциональный" подход, главная проблема в том, что нельзя передать заивисимости.
А вот так уже имеет смысл:
export const login = (authService) => (req, resp) => {
// do something
}
В целом, если у фреймворка/проекта есть устоявшийся подход - используем его. Если нет, то подход на функциях более идиоматический - в последние годы он более привычен большинству js-программистов, все почему-то двигаются в сторону функциональщины.