refactor: 스터디룸 파일 업로드 s3 + fileAttachment + Mapping 제거 방식으로 수정

loseminho · loseminho · commit e20e5f018507 · 2025-10-15T19:52:53.000+09:00
diff --git a/src/main/java/com/back/domain/studyroom/service/RoomService.java b/src/main/java/com/back/domain/studyroom/service/RoomService.java
@@ -346,9 +346,9 @@ public void updateRoomSettings(Long roomId, String title, String description,
         // 썸네일 변경 처리
         String thumbnailUrl = room.getRawThumbnailUrl();  // 기존 URL 유지
         if (thumbnailAttachmentId != null) {
-            // 기존 매핑 삭제 + 새 매핑 생성
+            // 기존 매핑 삭제 + 새 매핑 생성 (S3 파일도 삭제)
             thumbnailUrl = roomThumbnailService.updateThumbnailMapping(
-                    roomId, thumbnailAttachmentId);
+                    roomId, thumbnailAttachmentId, userId);
         }
 
         room.updateSettings(title, description, maxParticipants, thumbnailUrl);
@@ -448,8 +448,8 @@ public void terminateRoom(Long roomId, Long userId) {
             throw new CustomException(ErrorCode.NOT_ROOM_MANAGER);
         }
 
-        // 썸네일 매핑 삭제
-        roomThumbnailService.deleteThumbnailMapping(roomId);
+        // 썸네일 매핑 삭제 (S3 파일 + FileAttachment + Mapping)
+        roomThumbnailService.deleteThumbnailMapping(roomId, userId);
 
         room.terminate();
         
diff --git a/src/main/java/com/back/domain/studyroom/service/RoomThumbnailService.java b/src/main/java/com/back/domain/studyroom/service/RoomThumbnailService.java
@@ -5,6 +5,7 @@
 import com.back.domain.file.entity.FileAttachment;
 import com.back.domain.file.repository.AttachmentMappingRepository;
 import com.back.domain.file.repository.FileAttachmentRepository;
+import com.back.domain.file.service.FileService;
 import com.back.global.exception.CustomException;
 import com.back.global.exception.ErrorCode;
 import lombok.RequiredArgsConstructor;
@@ -25,6 +26,7 @@ public class RoomThumbnailService {
 
     private final FileAttachmentRepository fileAttachmentRepository;
     private final AttachmentMappingRepository attachmentMappingRepository;
+    private final FileService fileService;
 
     /**
      * 방 생성 시 썸네일 매핑 생성
@@ -60,42 +62,83 @@ public String createThumbnailMapping(Long roomId, Long thumbnailAttachmentId) {
 
     /**
      * 방 수정 시 썸네일 변경
-     * 1. 기존 매핑 삭제
+     * 1. 기존 매핑 및 파일 삭제 (S3 + FileAttachment + Mapping)
      * 2. 새 매핑 생성
      * 
      * @param roomId 방 ID
      * @param newThumbnailAttachmentId 새 썸네일 파일 ID (null이면 변경 없음)
+     * @param userId 요청자 ID (파일 삭제 권한 검증용)
      * @return 새 썸네일 URL (null이면 변경 없음)
      */
     @Transactional
-    public String updateThumbnailMapping(Long roomId, Long newThumbnailAttachmentId) {
+    public String updateThumbnailMapping(Long roomId, Long newThumbnailAttachmentId, Long userId) {
         if (newThumbnailAttachmentId == null) {
             // null이면 썸네일 변경 없음
             return null;
         }
 
-        // 기존 매핑 모두 삭제
+        // 기존 매핑 및 파일 삭제 (S3 + FileAttachment 포함)
+        List<AttachmentMapping> mappings = attachmentMappingRepository
+                .findAllByEntityTypeAndEntityId(EntityType.STUDY_ROOM, roomId);
+        
+        for (AttachmentMapping mapping : mappings) {
+            FileAttachment oldAttachment = mapping.getFileAttachment();
+            if (oldAttachment != null) {
+                // FileService를 사용하여 S3 파일 + FileAttachment 삭제
+                try {
+                    fileService.deleteFile(oldAttachment.getId(), userId);
+                    log.info("기존 썸네일 파일 삭제 - RoomId: {}, AttachmentId: {}", 
+                            roomId, oldAttachment.getId());
+                } catch (Exception e) {
+                    log.error("썸네일 파일 삭제 실패 - RoomId: {}, AttachmentId: {}, Error: {}", 
+                            roomId, oldAttachment.getId(), e.getMessage());
+                }
+            }
+        }
+        
+        // 매핑 삭제
         attachmentMappingRepository.deleteAllByEntityTypeAndEntityId(
                 EntityType.STUDY_ROOM, roomId);
         
-        log.info("기존 썸네일 매핑 삭제 - RoomId: {}", roomId);
+        log.info("기존 썸네일 매핑 및 파일 삭제 완료 - RoomId: {}", roomId);
 
         // 새 매핑 생성
         return createThumbnailMapping(roomId, newThumbnailAttachmentId);
     }
 
     /**
-     * 방 삭제 시 썸네일 매핑 삭제
+     * 방 삭제 시 썸네일 매핑 및 파일 삭제
+     * S3 파일 + FileAttachment + AttachmentMapping 모두 삭제
      * 
      * @param roomId 방 ID
+     * @param userId 요청자 ID (파일 삭제 권한 검증용)
      */
     @Transactional
-    public void deleteThumbnailMapping(Long roomId) {
-        // 연결된 파일 매핑 모두 삭제
+    public void deleteThumbnailMapping(Long roomId, Long userId) {
+        // 매핑 조회
+        List<AttachmentMapping> mappings = attachmentMappingRepository
+                .findAllByEntityTypeAndEntityId(EntityType.STUDY_ROOM, roomId);
+        
+        // S3 파일 + FileAttachment 삭제
+        for (AttachmentMapping mapping : mappings) {
+            FileAttachment fileAttachment = mapping.getFileAttachment();
+            if (fileAttachment != null) {
+                try {
+                    fileService.deleteFile(fileAttachment.getId(), userId);
+                    log.info("썸네일 파일 삭제 완료 - RoomId: {}, AttachmentId: {}", 
+                            roomId, fileAttachment.getId());
+                } catch (Exception e) {
+                    log.error("썸네일 파일 삭제 실패 - RoomId: {}, AttachmentId: {}, Error: {}", 
+                            roomId, fileAttachment.getId(), e.getMessage());
+                }
+            }
+        }
+        
+        // AttachmentMapping 삭제
         attachmentMappingRepository.deleteAllByEntityTypeAndEntityId(
                 EntityType.STUDY_ROOM, roomId);
         
-        log.info("방 삭제 - 썸네일 매핑 삭제 완료 - RoomId: {}", roomId);
+        log.info("방 삭제 - 썸네일 매핑 및 파일 삭제 완료 - RoomId: {}", roomId);
     }
 
     /**
diff --git a/src/main/resources/application.yml b/src/main/resources/application.yml
@@ -123,8 +123,8 @@ webrtc:
     - urls: stun:stun1.l.google.com:19302
     - urls: stun:stun2.l.google.com:19302
   turn:
-    shared-secret: "${WEBRTC_TURN_SHARED_SECRET}"
-    server-ip: "${WEBRTC_TURN_SERVER_IP}"
+    shared-secret: "${WEBRTC_TURN_SHARED_SECRET:default-secret-key}"
+    server-ip: "${WEBRTC_TURN_SERVER_IP:127.0.0.1}"
     ttl-seconds: 3600
 
 # 스터디룸 설정
diff --git a/src/test/java/com/back/domain/studyroom/service/RoomServiceTest.java b/src/test/java/com/back/domain/studyroom/service/RoomServiceTest.java
@@ -313,7 +313,7 @@ void updateRoomSettings_Success() {
         assertThat(testRoom.getTitle()).isEqualTo("변경된 제목");
         assertThat(testRoom.getDescription()).isEqualTo("변경된 설명");
         assertThat(testRoom.getMaxParticipants()).isEqualTo(15);
-        verify(roomThumbnailService, never()).updateThumbnailMapping(any(), any());
+        verify(roomThumbnailService, never()).updateThumbnailMapping(any(), any(), any());  // userId 파라미터 추가
     }
 
     @Test
@@ -348,7 +348,7 @@ void terminateRoom_Success() {
         // then
         assertThat(testRoom.getStatus()).isEqualTo(RoomStatus.TERMINATED);
         assertThat(testRoom.isActive()).isFalse();
-        verify(roomThumbnailService, times(1)).deleteThumbnailMapping(1L);
+        verify(roomThumbnailService, times(1)).deleteThumbnailMapping(1L, 1L);  // userId 파라미터 추가
     }
 
     @Test
@@ -617,7 +617,7 @@ void updateRoomSettings_WithThumbnailChange() {
         // given
         given(roomRepository.findById(1L)).willReturn(Optional.of(testRoom));
         given(roomParticipantService.getParticipantCount(1L)).willReturn(0L);
-        given(roomThumbnailService.updateThumbnailMapping(eq(1L), eq(789L)))
+        given(roomThumbnailService.updateThumbnailMapping(eq(1L), eq(789L), eq(1L)))  // userId 파라미터 추가
                 .willReturn("https://s3.amazonaws.com/bucket/new-thumbnail.jpg");
 
         // when
@@ -633,6 +633,6 @@ void updateRoomSettings_WithThumbnailChange() {
         // then
         assertThat(testRoom.getTitle()).isEqualTo("변경된 제목");
         assertThat(testRoom.getThumbnailUrl()).isEqualTo("https://s3.amazonaws.com/bucket/new-thumbnail.jpg");
-        verify(roomThumbnailService, times(1)).updateThumbnailMapping(eq(1L), eq(789L));
+        verify(roomThumbnailService, times(1)).updateThumbnailMapping(eq(1L), eq(789L), eq(1L));  // userId 파라미터 추가
     }
 }
