У меня имеется эндпоинт /reports
. Пользователь с ролью STUDENT делает запрос на данный эндпоинт и получает ответ. Изначально соответствующий метод в rest-контроллере выглядел вот так:
@GetMapping("/reports")
public ReportsTo getAll(@AuthenticationPrincipal AuthorizedUser user) {
return service.getToByStudentId(user.getProfile().getId());
}
Я добавил роль TUTOR (преподаватель). Пользователь с такой ролью тоже имеет доступ к методу getAll()
. Как вы могли видеть, в предыдущем варианте я получал id студента из принципала, но теперь, когда я расширяю метод getAll()
, такой способ уже не будет работать (так как преподаватель содержит свой id, а не студента), и теперь метод выглядит вот так:
@GetMapping("/reports")
public ReportsTo getAll(@RequestParam(value = "student-id") Integer studentId,
@RequestParam(value = "theme-id", required = false) Integer themeId,
@AuthenticationPrincipal AuthorizedUser user) {
return switch (user.getProfile().getRole()) {
case STUDENT -> {
checkIdCompliance(studentId, user.getProfile());
yield service.getToByStudentIdAndThemeId(studentId, themeId);
}
case TUTOR -> {
if (themeId == null) {
throw new ResourceIsNotAllowedException("Theme id can not be null");
}
yield service.getToByStudentId(studentId, themeId, pagination);
}
default -> throw new ResourceIsNotAllowedException(
"This resource is available only for users with the following roles: STUDENT, TUTOR");
};
}
Обратите внимание на параметры. Теперь мне приходится передавать в метод очень много лишних параметров. Например themeId
не нужен студенту, как не нужен и studentId
. В свою очередь для преподавателя лишним является получение принципала.
Я вижу здесь только одно решение: сделать два метода на один и тот же эндпоинт. Я представляю это вот так:
@GetMapping("/reports")
public ReportsTo getAll(@AuthenticationPrincipal AuthorizedUser user) {
return service.getToByStudentId(user.getProfile().getId());
}
@GetMapping("/reports")
public ReportsTo getAll(@RequestParam(value = "student-id") Integer studentId,
@RequestParam(value = "theme-id") Integer themeId) {
return service.getToByStudentIdAndThemeId(studentId, themeId);
}
Вот только это не будет работать, потому что Spring выдаст ошибку при наличии двух одинаковых эндпоинтов.
Проблема: разные роли могут иметь доступ к одному и тому же эндпоинту. При этом эндпоинт будет засоряться множеством параметров и проверок. Как эту проблему решить?