diff --git a/docs/design/494-delete-impact-confirmation/README.md b/docs/design/494-delete-impact-confirmation/README.md new file mode 100644 index 0000000..07da082 --- /dev/null +++ b/docs/design/494-delete-impact-confirmation/README.md @@ -0,0 +1,56 @@ +# Redmine #494 - 삭제 영향도와 확인 절차 설계 + +## 프로젝트 개요 + +VPD Backoffice는 Oracle Database VPD/ORDS 기능을 백오피스 권한 테이블로 제어하는 Spring Boot 관리 도구다. 사용자는 DB schema user가 아니라 Bearer Token으로 식별되는 application user이며, 사용자/그룹/역할/권한 테이블의 조합이 VPD policy function과 redaction 판단에 사용된다. + +## 목표 + +권한, 역할, 그룹 관련 삭제/해제/비활성화 작업 전에 영향 범위를 보여주고, 사용자가 영향 확인을 명시한 경우에만 처리한다. + +## 현재 문제 + +- 역할 삭제 버튼은 연결 사용자, 그룹, 권한이 남아 있어도 영향 범위를 보여주지 않는다. +- 보조 스키마 생성 DDL에는 FK가 없어 연결이 남은 역할을 삭제하면 고아 매핑/권한이 생길 수 있다. +- 권한 삭제 시 해당 역할의 TABLE/VIEW 접근이 사라지고 마지막 권한이면 보호 객체가 비활성화될 수 있지만 화면에서 강조하지 않는다. +- 그룹 비활성화/그룹 사용자 해제/그룹 역할 해제는 VPD effective role 결과를 바꾸지만 확인 절차가 약하다. +- 삭제 검증 실패가 전역 오류 화면으로 빠져 사용자가 원래 화면에서 바로 조치하기 어렵다. + +## 설계 + +- 역할 목록에 삭제 영향 요약을 표시한다. + - 직접 사용자 + - 연결 그룹 + - 영향 사용자 + - 권한 수와 보호 객체 +- 역할 삭제는 서버에서 연결 사용자/그룹/권한이 하나라도 있으면 차단한다. + - 삭제 전에 사용자 역할, 그룹 역할, 권한을 먼저 해제해야 한다. + - 연결이 없는 역할도 영향 확인 체크가 있어야 삭제된다. +- 권한 목록에 삭제 영향 요약을 표시한다. + - 역할이 잃는 TABLE/VIEW 권한 + - 마지막 권한 삭제 시 보호 객체 비활성화 가능성 + - 영향 확인 체크가 없으면 서버에서 차단한다. +- 그룹 화면에 비활성화/해제 영향 요약을 표시한다. + - 그룹 비활성화는 활성 그룹을 통해 상속되던 역할을 VPD에서 제외한다. + - 그룹 사용자 해제는 해당 사용자가 그 그룹 역할을 잃는다. + - 그룹 역할 해제는 그룹 사용자 전체가 해당 역할 권한을 잃는다. + - 영향 확인 체크가 없으면 서버에서 차단한다. +- 삭제/해제 차단 메시지는 전역 오류 페이지 대신 원래 화면의 error alert로 보여준다. + +## 완료 기준 + +- 역할 목록에서 삭제 영향이 보이고 연결이 있는 역할 삭제는 차단된다. +- 권한 목록에서 삭제 영향과 마지막 권한 경고가 보인다. +- 권한 삭제는 영향 확인 체크 없이는 서버에서 실패한다. +- 그룹 비활성화/사용자 해제/역할 해제는 영향 확인 체크를 요구한다. +- 실패 메시지는 원래 화면 상단에서 확인된다. +- 모바일 화면에서 가로 body overflow가 없다. + +## 검증 + +- `mvn test` +- Playwright: + - `/roles`, `/permissions`, `/groups` desktop/mobile 렌더링 + - 영향 확인 UI 노출 확인 + - 확인 체크 없는 권한 삭제 시 error alert 확인 + - 모바일 body overflow 없음 diff --git a/src/main/java/com/cloudhandson/vpdbackoffice/mapper/PermissionMapper.java b/src/main/java/com/cloudhandson/vpdbackoffice/mapper/PermissionMapper.java index 53f18ba..46e69d6 100644 --- a/src/main/java/com/cloudhandson/vpdbackoffice/mapper/PermissionMapper.java +++ b/src/main/java/com/cloudhandson/vpdbackoffice/mapper/PermissionMapper.java @@ -27,6 +27,12 @@ public interface PermissionMapper { int deleteRole(@Param("roleId") long roleId); + int countUserRolesByRoleId(@Param("roleId") long roleId); + + int countGroupRolesByRoleId(@Param("roleId") long roleId); + + int countPermissionsByRoleId(@Param("roleId") long roleId); + List findPermissionViews(); PermissionSet findPermissionSet(@Param("roleId") long roleId, @Param("objectId") long objectId); diff --git a/src/main/java/com/cloudhandson/vpdbackoffice/service/GroupService.java b/src/main/java/com/cloudhandson/vpdbackoffice/service/GroupService.java index fe18013..78614b5 100644 --- a/src/main/java/com/cloudhandson/vpdbackoffice/service/GroupService.java +++ b/src/main/java/com/cloudhandson/vpdbackoffice/service/GroupService.java @@ -42,6 +42,14 @@ public class GroupService { @Transactional public void setActive(long groupId, boolean active) { + setActive(groupId, active, true); + } + + @Transactional + public void setActive(long groupId, boolean active, boolean confirmImpact) { + if (!active && !confirmImpact) { + throw new AppException("그룹 비활성화 전 영향 확인이 필요합니다."); + } int updated = groupMapper.updateActive(groupId, active ? "Y" : "N"); if (updated == 0) { throw new AppException("그룹을 찾을 수 없습니다."); @@ -59,6 +67,14 @@ public class GroupService { @Transactional public void removeUser(long groupId, long userId) { + removeUser(groupId, userId, true); + } + + @Transactional + public void removeUser(long groupId, long userId, boolean confirmImpact) { + if (!confirmImpact) { + throw new AppException("그룹 사용자 해제 전 영향 확인이 필요합니다."); + } int deleted = groupMapper.deleteGroupUser(groupId, userId); if (deleted == 0) { throw new AppException("삭제할 그룹 사용자 매핑을 찾을 수 없습니다."); @@ -76,6 +92,14 @@ public class GroupService { @Transactional public void removeRole(long groupId, long roleId) { + removeRole(groupId, roleId, true); + } + + @Transactional + public void removeRole(long groupId, long roleId, boolean confirmImpact) { + if (!confirmImpact) { + throw new AppException("그룹 역할 해제 전 영향 확인이 필요합니다."); + } int deleted = groupMapper.deleteGroupRole(groupId, roleId); if (deleted == 0) { throw new AppException("삭제할 그룹 역할 매핑을 찾을 수 없습니다."); diff --git a/src/main/java/com/cloudhandson/vpdbackoffice/service/PermissionService.java b/src/main/java/com/cloudhandson/vpdbackoffice/service/PermissionService.java index dc5e592..fa8a57f 100644 --- a/src/main/java/com/cloudhandson/vpdbackoffice/service/PermissionService.java +++ b/src/main/java/com/cloudhandson/vpdbackoffice/service/PermissionService.java @@ -76,6 +76,22 @@ public class PermissionService { @Transactional public void deleteRole(long roleId) { + deleteRole(roleId, true); + } + + @Transactional + public void deleteRole(long roleId, boolean confirmImpact) { + if (!confirmImpact) { + throw new AppException("역할 삭제 전 영향 확인이 필요합니다."); + } + int userRoleCount = permissionMapper.countUserRolesByRoleId(roleId); + int groupRoleCount = permissionMapper.countGroupRolesByRoleId(roleId); + int permissionCount = permissionMapper.countPermissionsByRoleId(roleId); + if (userRoleCount + groupRoleCount + permissionCount > 0) { + throw new AppException("연결된 사용자/그룹/권한이 있는 역할은 삭제할 수 없습니다. 사용자 역할 " + + userRoleCount + "건, 그룹 역할 " + groupRoleCount + "건, 권한 " + permissionCount + + "건을 먼저 해제하세요."); + } int deleted = permissionMapper.deleteRole(roleId); if (deleted == 0) { throw new AppException("삭제할 역할을 찾을 수 없습니다."); @@ -133,6 +149,14 @@ public class PermissionService { @Transactional public void deletePermission(long permissionId) { + deletePermission(permissionId, true); + } + + @Transactional + public void deletePermission(long permissionId, boolean confirmImpact) { + if (!confirmImpact) { + throw new AppException("권한 삭제 전 영향 확인이 필요합니다."); + } Long objectId = permissionMapper.findObjectIdByPermissionId(permissionId); permissionMapper.deleteRules(permissionId); permissionMapper.deleteVisibleColumns(permissionId); @@ -147,6 +171,10 @@ public class PermissionService { "permissionId=" + permissionId)); } + public int countPermissionsByObjectId(long objectId) { + return permissionMapper.countPermissionsByObjectId(objectId); + } + private void validateRules(long objectId, List rules) { if (rules == null || rules.isEmpty()) { throw new AppException("행 규칙은 하나 이상 필요합니다."); diff --git a/src/main/java/com/cloudhandson/vpdbackoffice/web/GroupController.java b/src/main/java/com/cloudhandson/vpdbackoffice/web/GroupController.java index a8e74f5..a5175e1 100644 --- a/src/main/java/com/cloudhandson/vpdbackoffice/web/GroupController.java +++ b/src/main/java/com/cloudhandson/vpdbackoffice/web/GroupController.java @@ -1,9 +1,15 @@ package com.cloudhandson.vpdbackoffice.web; +import com.cloudhandson.vpdbackoffice.domain.effective.GroupEffectiveAccessView; +import com.cloudhandson.vpdbackoffice.domain.effective.RoleEffectiveImpactView; import com.cloudhandson.vpdbackoffice.domain.group.GroupCreateCommand; +import com.cloudhandson.vpdbackoffice.service.AppException; +import com.cloudhandson.vpdbackoffice.service.EffectiveMatrixService; import com.cloudhandson.vpdbackoffice.service.GroupService; import com.cloudhandson.vpdbackoffice.service.PermissionService; import com.cloudhandson.vpdbackoffice.service.UserService; +import java.util.Map; +import java.util.stream.Collectors; import org.springframework.stereotype.Controller; import org.springframework.ui.Model; import org.springframework.web.bind.annotation.GetMapping; @@ -17,20 +23,32 @@ public class GroupController { private final GroupService groupService; private final UserService userService; private final PermissionService permissionService; + private final EffectiveMatrixService effectiveMatrixService; - public GroupController(GroupService groupService, UserService userService, PermissionService permissionService) { + public GroupController( + GroupService groupService, + UserService userService, + PermissionService permissionService, + EffectiveMatrixService effectiveMatrixService + ) { this.groupService = groupService; this.userService = userService; this.permissionService = permissionService; + this.effectiveMatrixService = effectiveMatrixService; } @GetMapping("/groups") public String groups(Model model) { + var matrix = effectiveMatrixService.matrix(); model.addAttribute("groups", groupService.findAll()); model.addAttribute("users", userService.findAll()); model.addAttribute("roles", permissionService.findRoles()); model.addAttribute("groupUsers", groupService.findGroupUsers()); model.addAttribute("groupRoles", groupService.findGroupRoles()); + model.addAttribute("groupImpactByGroupId", matrix.groups().stream() + .collect(Collectors.toMap(GroupEffectiveAccessView::groupId, impact -> impact))); + model.addAttribute("roleImpactByRoleId", matrix.roles().stream() + .collect(Collectors.toMap(RoleEffectiveImpactView::roleId, impact -> impact))); return "groups"; } @@ -50,10 +68,15 @@ public class GroupController { public String active( @RequestParam long groupId, @RequestParam boolean active, + @RequestParam(defaultValue = "false") boolean confirmImpact, RedirectAttributes redirectAttributes ) { - groupService.setActive(groupId, active); - redirectAttributes.addFlashAttribute("message", "그룹 상태를 변경했습니다."); + try { + groupService.setActive(groupId, active, active || confirmImpact); + redirectAttributes.addFlashAttribute("message", "그룹 상태를 변경했습니다."); + } catch (AppException exception) { + redirectAttributes.addFlashAttribute("error", exception.getMessage()); + } return "redirect:/groups"; } @@ -72,10 +95,15 @@ public class GroupController { public String removeUser( @RequestParam long groupId, @RequestParam long userId, + @RequestParam(defaultValue = "false") boolean confirmImpact, RedirectAttributes redirectAttributes ) { - groupService.removeUser(groupId, userId); - redirectAttributes.addFlashAttribute("message", "그룹 사용자를 해제했습니다."); + try { + groupService.removeUser(groupId, userId, confirmImpact); + redirectAttributes.addFlashAttribute("message", "그룹 사용자를 해제했습니다."); + } catch (AppException exception) { + redirectAttributes.addFlashAttribute("error", exception.getMessage()); + } return "redirect:/groups"; } @@ -94,10 +122,15 @@ public class GroupController { public String removeRole( @RequestParam long groupId, @RequestParam long roleId, + @RequestParam(defaultValue = "false") boolean confirmImpact, RedirectAttributes redirectAttributes ) { - groupService.removeRole(groupId, roleId); - redirectAttributes.addFlashAttribute("message", "그룹 역할을 해제했습니다."); + try { + groupService.removeRole(groupId, roleId, confirmImpact); + redirectAttributes.addFlashAttribute("message", "그룹 역할을 해제했습니다."); + } catch (AppException exception) { + redirectAttributes.addFlashAttribute("error", exception.getMessage()); + } return "redirect:/groups"; } } diff --git a/src/main/java/com/cloudhandson/vpdbackoffice/web/PermissionController.java b/src/main/java/com/cloudhandson/vpdbackoffice/web/PermissionController.java index f2e56db..01bc6e5 100644 --- a/src/main/java/com/cloudhandson/vpdbackoffice/web/PermissionController.java +++ b/src/main/java/com/cloudhandson/vpdbackoffice/web/PermissionController.java @@ -4,6 +4,7 @@ import com.cloudhandson.vpdbackoffice.domain.permission.AppRole; import com.cloudhandson.vpdbackoffice.domain.permission.PermissionSetCommand; import com.cloudhandson.vpdbackoffice.domain.permission.RuleCommand; import com.cloudhandson.vpdbackoffice.domain.protectedobject.ProtectedColumn; +import com.cloudhandson.vpdbackoffice.service.AppException; import com.cloudhandson.vpdbackoffice.service.GroupService; import com.cloudhandson.vpdbackoffice.service.PermissionService; import com.cloudhandson.vpdbackoffice.service.ProtectedObjectService; @@ -111,6 +112,14 @@ public class PermissionController { var dbObjects = protectedObjectService.findDatabaseObjects(); long dbObjectsAt = System.nanoTime(); var permissions = permissionService.findPermissionViews(); + var lastPermissionByPermissionId = permissions.stream() + .collect(Collectors.toMap( + permission -> permission.permissionId(), + permission -> permission.objectId() > 0 + && permissionService.countPermissionsByObjectId(permission.objectId()) <= 1, + (left, right) -> left, + LinkedHashMap::new + )); long permissionsAt = System.nanoTime(); log.info("permissions page timings: objects={}ms roles={}ms roleImpact={}ms columns={}ms dbObjects={}ms permissions={}ms total={}ms", elapsedMillis(started, objectsAt), @@ -130,6 +139,7 @@ public class PermissionController { model.addAttribute("groupUsersByRole", roleImpact.groupUsersByRole()); model.addAttribute("dbObjects", dbObjects); model.addAttribute("permissions", permissions); + model.addAttribute("lastPermissionByPermissionId", lastPermissionByPermissionId); return "permissions"; } @@ -257,9 +267,17 @@ public class PermissionController { } @PostMapping("/permissions/delete") - public String delete(@RequestParam long permissionId, RedirectAttributes redirectAttributes) { - permissionService.deletePermission(permissionId); - redirectAttributes.addFlashAttribute("message", "권한을 삭제했습니다."); + public String delete( + @RequestParam long permissionId, + @RequestParam(defaultValue = "false") boolean confirmImpact, + RedirectAttributes redirectAttributes + ) { + try { + permissionService.deletePermission(permissionId, confirmImpact); + redirectAttributes.addFlashAttribute("message", "권한을 삭제했습니다."); + } catch (AppException exception) { + redirectAttributes.addFlashAttribute("error", exception.getMessage()); + } return "redirect:/permissions"; } } diff --git a/src/main/java/com/cloudhandson/vpdbackoffice/web/RoleController.java b/src/main/java/com/cloudhandson/vpdbackoffice/web/RoleController.java index 04769a8..079fe61 100644 --- a/src/main/java/com/cloudhandson/vpdbackoffice/web/RoleController.java +++ b/src/main/java/com/cloudhandson/vpdbackoffice/web/RoleController.java @@ -1,6 +1,11 @@ package com.cloudhandson.vpdbackoffice.web; +import com.cloudhandson.vpdbackoffice.domain.effective.RoleEffectiveImpactView; +import com.cloudhandson.vpdbackoffice.service.AppException; +import com.cloudhandson.vpdbackoffice.service.EffectiveMatrixService; import com.cloudhandson.vpdbackoffice.service.PermissionService; +import java.util.Map; +import java.util.stream.Collectors; import org.springframework.stereotype.Controller; import org.springframework.ui.Model; import org.springframework.web.bind.annotation.GetMapping; @@ -12,14 +17,19 @@ import org.springframework.web.servlet.mvc.support.RedirectAttributes; public class RoleController { private final PermissionService permissionService; + private final EffectiveMatrixService effectiveMatrixService; - public RoleController(PermissionService permissionService) { + public RoleController(PermissionService permissionService, EffectiveMatrixService effectiveMatrixService) { this.permissionService = permissionService; + this.effectiveMatrixService = effectiveMatrixService; } @GetMapping("/roles") public String roles(Model model) { model.addAttribute("roles", permissionService.findRoles()); + Map roleImpactByRoleId = effectiveMatrixService.matrix().roles().stream() + .collect(Collectors.toMap(RoleEffectiveImpactView::roleId, impact -> impact)); + model.addAttribute("roleImpactByRoleId", roleImpactByRoleId); return "roles"; } @@ -47,9 +57,17 @@ public class RoleController { } @PostMapping("/roles/delete") - public String delete(@RequestParam long roleId, RedirectAttributes redirectAttributes) { - permissionService.deleteRole(roleId); - redirectAttributes.addFlashAttribute("message", "역할을 삭제했습니다."); + public String delete( + @RequestParam long roleId, + @RequestParam(defaultValue = "false") boolean confirmImpact, + RedirectAttributes redirectAttributes + ) { + try { + permissionService.deleteRole(roleId, confirmImpact); + redirectAttributes.addFlashAttribute("message", "역할을 삭제했습니다."); + } catch (AppException exception) { + redirectAttributes.addFlashAttribute("error", exception.getMessage()); + } return "redirect:/roles"; } } diff --git a/src/main/resources/mapper/PermissionMapper.xml b/src/main/resources/mapper/PermissionMapper.xml index 0a016d6..4afad79 100644 --- a/src/main/resources/mapper/PermissionMapper.xml +++ b/src/main/resources/mapper/PermissionMapper.xml @@ -44,6 +44,24 @@ WHERE role_id = #{roleId,jdbcType=NUMERIC} + + + + + + + - 등록된 그룹이 없습니다. + 등록된 그룹이 없습니다. @@ -95,24 +112,33 @@ 그룹 사용자 + 해제 영향 - + SALES_TEAM agent_sales + + 이 사용자는 그룹 역할을 더 이상 상속하지 않습니다. + 그룹 역할 없음 +
+
- 등록된 그룹 사용자가 없습니다. + 등록된 그룹 사용자가 없습니다. @@ -145,24 +171,35 @@ 그룹 역할 + 해제 영향 - + SALES_TEAM SALES_ROLE + + 그룹 사용자가 이 역할 권한을 잃습니다. + 영향 사용자 없음 + 권한 객체 없음 +
+
- 등록된 그룹 역할이 없습니다. + 등록된 그룹 역할이 없습니다. diff --git a/src/main/resources/templates/permissions.html b/src/main/resources/templates/permissions.html index caf1b07..a090482 100644 --- a/src/main/resources/templates/permissions.html +++ b/src/main/resources/templates/permissions.html @@ -12,6 +12,7 @@
+
@@ -222,11 +223,13 @@ 행 규칙 적용 필터 원문 표시 허용 컬럼 + 삭제 영향 - + 100 HR_DEPT_ROLE CB_V_SEARCH_DOCUMENTS @@ -239,16 +242,26 @@ ALL 
ALL ROWS
CONTENTS + + 역할이 이 TABLE/VIEW의 SELECT 권한을 잃습니다. + ROLE -> OBJECT + 이 객체의 마지막 권한입니다. 삭제 후 보호 객체가 비활성화됩니다. + 같은 객체에 다른 역할 권한이 남아 있습니다. +
+
- 등록된 권한이 없습니다. + 등록된 권한이 없습니다. diff --git a/src/main/resources/templates/roles.html b/src/main/resources/templates/roles.html index f0adc87..6d99510 100644 --- a/src/main/resources/templates/roles.html +++ b/src/main/resources/templates/roles.html @@ -10,6 +10,7 @@
+

역할 추가

@@ -45,11 +46,14 @@ ID 역할명 민감도 허용 상한 + 삭제 영향 - + 10 HR_DEPT_ROLE @@ -65,16 +69,42 @@ + +
+ 직접 사용자 + 없음 +
+
+ 연결 그룹 + 없음 +
+
+ 영향 사용자 + 없음 +
+
+ 권한/객체 + 0건 + 보호 객체 없음 +
+

연결을 먼저 해제해야 역할을 삭제할 수 있습니다.

+
- + +
- 등록된 역할이 없습니다. + 등록된 역할이 없습니다. diff --git a/src/test/java/com/cloudhandson/vpdbackoffice/service/PermissionServiceTest.java b/src/test/java/com/cloudhandson/vpdbackoffice/service/PermissionServiceTest.java index b280281..743f9d5 100644 --- a/src/test/java/com/cloudhandson/vpdbackoffice/service/PermissionServiceTest.java +++ b/src/test/java/com/cloudhandson/vpdbackoffice/service/PermissionServiceTest.java @@ -173,6 +173,24 @@ class PermissionServiceTest { assertThat(disabled[0]).isTrue(); } + @Test + void rejectsPermissionDeleteWithoutImpactConfirmation() { + assertThatThrownBy(() -> permissionService.deletePermission(1000L, false)) + .isInstanceOf(AppException.class) + .hasMessageContaining("영향 확인"); + } + + @Test + void blocksRoleDeleteWhenDependenciesRemain() { + var mapper = new FakePermissionMapper(); + mapper.userRoleCount = 1; + PermissionService service = new PermissionService(mapper, protectedObjectService, new AuditService(new NoopAuditMapper())); + + assertThatThrownBy(() -> service.deleteRole(10L, true)) + .isInstanceOf(AppException.class) + .hasMessageContaining("먼저 해제"); + } + private static class NoopAuditMapper implements AuditMapper { @Override public void insert(AuditEvent event) { @@ -211,6 +229,25 @@ class PermissionServiceTest { return 1; } + private int userRoleCount; + private int groupRoleCount; + private int permissionCountByRole; + + @Override + public int countUserRolesByRoleId(long roleId) { + return userRoleCount; + } + + @Override + public int countGroupRolesByRoleId(long roleId) { + return groupRoleCount; + } + + @Override + public int countPermissionsByRoleId(long roleId) { + return permissionCountByRole; + } + @Override public List findPermissionViews() { return List.of();