영상 파이프라인 중 메타데이터 저장 (#102)

tlswltjq · web-flow · commit e8a8d4cd2e15 · 2025-09-29T12:51:32.000+09:00
* Feat : 조회수 추가기능 작성

* Feat : Operation 추가

* Feat : 카프카 의존성 추가 및 설정

* Refactor : 파일의 크기는 status에 포함되므로 제거

* Refactor : 응답 String으로 변경

* Feat : 트랜스코딩 메시지 프로듀싱

* Feat : 메타데이터 저장
diff --git a/back/build.gradle.kts b/back/build.gradle.kts
@@ -66,6 +66,8 @@ dependencies {
 	testRuntimeOnly("org.junit.platform:junit-platform-launcher")
 
 	implementation ("software.amazon.awssdk:s3:2.25.0")
+
+	implementation ("org.springframework.kafka:spring-kafka")
 }
 
 tasks.withType<Test> {
diff --git a/back/src/main/java/com/back/domain/file/video/controller/VideoController.java b/back/src/main/java/com/back/domain/file/video/controller/VideoController.java
@@ -4,6 +4,7 @@
 import com.back.domain.file.video.dto.service.PresignedUrlResponse;
 import com.back.domain.file.video.service.FileManager;
 import com.back.global.rsData.RsData;
+import io.swagger.v3.oas.annotations.Operation;
 import lombok.RequiredArgsConstructor;
 import org.springframework.web.bind.annotation.GetMapping;
 import org.springframework.web.bind.annotation.RequestParam;
@@ -15,16 +16,18 @@ public class VideoController {
     private final FileManager fileManager;
 
     @GetMapping("/videos/upload")
+    @Operation(summary="업로드용 URL 요청", description="파일 업로드를 위한 Presigned URL을 발급받습니다.")
     public RsData<UploadUrlGetResponse> getUploadUrl() {
         PresignedUrlResponse uploadUrl = fileManager.getUploadUrl();
-        UploadUrlGetResponse response = new UploadUrlGetResponse(uploadUrl.url(), uploadUrl.expiresAt());
+        UploadUrlGetResponse response = new UploadUrlGetResponse(uploadUrl.url().toString(), uploadUrl.expiresAt());
         return new RsData<>("200", "업로드용 URL 요청완료", response);
     }
 
     @GetMapping("/videos/download")
+    @Operation(summary="다운로드용 URL 요청", description="파일 다운로드를 위한 Presigned URL을 발급받습니다.")
     public RsData<UploadUrlGetResponse> getDownloadUrls(@RequestParam String objectKey) {
         PresignedUrlResponse downloadUrl = fileManager.getDownloadUrl(objectKey);
-        UploadUrlGetResponse response = new UploadUrlGetResponse(downloadUrl.url(), downloadUrl.expiresAt());
+        UploadUrlGetResponse response = new UploadUrlGetResponse(downloadUrl.url().toString(), downloadUrl.expiresAt());
         return new RsData<>("200", "다운로드용 URL 요청완료", response);
     }
 }
diff --git a/back/src/main/java/com/back/domain/file/video/dto/controller/UploadUrlGetResponse.java b/back/src/main/java/com/back/domain/file/video/dto/controller/UploadUrlGetResponse.java
@@ -1,10 +1,9 @@
 package com.back.domain.file.video.dto.controller;
 
-import java.net.URL;
 import java.time.LocalDateTime;
 
 public record UploadUrlGetResponse(
-        URL url,
+        String url,
         LocalDateTime expiresAt
 ) {
 }
diff --git a/back/src/main/java/com/back/domain/file/video/entity/Video.java b/back/src/main/java/com/back/domain/file/video/entity/Video.java
@@ -24,18 +24,15 @@ public class Video extends BaseEntity {
 
     private Integer duration;
 
-    private Long fileSize;
-
     @Builder(access = AccessLevel.PRIVATE)
-    private Video(String uuid, String status, String path, Integer duration, Long fileSize) {
+    private Video(String uuid, String status, String path, Integer duration) {
         this.uuid = uuid;
         this.status = status;
         this.path = path;
         this.duration = duration;
-        this.fileSize = fileSize;
     }
 
-    public static Video create(String uuid, String status, String path, Integer duration, Long fileSize) {
+    public static Video create(String uuid, String status, String path, Integer duration) {
         if (uuid == null || uuid.isBlank()) {
             throw new IllegalArgumentException("uuid cannot be null or empty");
         }
@@ -48,7 +45,6 @@ public static Video create(String uuid, String status, String path, Integer dura
                 .status(status)
                 .path(path)
                 .duration(duration)
-                .fileSize(fileSize)
                 .build();
     }
     public void updateStatus(String status) {
diff --git a/back/src/main/java/com/back/domain/file/video/service/FileManager.java b/back/src/main/java/com/back/domain/file/video/service/FileManager.java
@@ -29,4 +29,13 @@ public PresignedUrlResponse getDownloadUrl(String objectKey) {
         LocalDateTime expiresAt = LocalDateTime.now().plusMinutes(expires);
         return new PresignedUrlResponse(url, expiresAt);
     }
+
+    //TODO : 테스트 작성필요
+    public void updateVideoStatus(String videoId, String status) {
+        try {
+            videoService.updateStatus(videoId, status);
+        } catch (Exception e) {
+            videoService.createVideo(videoId, status, "/", 0);
+        }
+    }
 }
diff --git a/back/src/main/java/com/back/domain/file/video/service/VideoService.java b/back/src/main/java/com/back/domain/file/video/service/VideoService.java
@@ -11,8 +11,8 @@
 public class VideoService {
     private final VideoRepository videoRepository;
 
-    public Video createVideo(String uuid, String status, String path, Integer duration, Long fileSize) {
-        Video video = Video.create(uuid, status, path, duration, fileSize);
+    public Video createVideo(String uuid, String status, String path, Integer duration) {
+        Video video = Video.create(uuid, status, path, duration);
         return videoRepository.save(video);
     }
 
diff --git a/back/src/main/java/com/back/domain/news/news/controller/NewsController.java b/back/src/main/java/com/back/domain/news/news/controller/NewsController.java
@@ -45,6 +45,7 @@ public RsData<NewsCreateResponse> createNews(@RequestBody NewsCreateRequest requ
     @Operation(summary = "뉴스 단건 조회", description = "특정 ID의 뉴스를 읽어옵니다.")
     public RsData<NewsGetResponse> getNews(@PathVariable("newsId") Long newsId) {
         News news = newsService.getNewsById(newsId);
+        newsService.incrementViews(news);
         NewsGetResponse response = new NewsGetResponse(news);
         return new RsData<>("200", "뉴스 읽어오기 완료", response);
     }
diff --git a/back/src/main/java/com/back/domain/news/news/entity/News.java b/back/src/main/java/com/back/domain/news/news/entity/News.java
@@ -24,17 +24,19 @@ public class News extends BaseEntity {
     @OneToOne
     private Video video;
     private String content;
+    private Integer views;
     @OneToMany(mappedBy = "news", cascade = CascadeType.ALL, orphanRemoval = true)
     private List<NewsComment> newsComment;
     @OneToMany(mappedBy = "news", cascade = CascadeType.ALL, orphanRemoval = true)
     private List<NewsLike> newsLikes = new ArrayList<>();
 
     @Builder(access = AccessLevel.PRIVATE)
-    private News(Member member, String title, Video video, String content, List<NewsComment> newsComment, List<NewsLike> newsLikes) {
+    private News(Member member, String title, Video video, String content, Integer views, List<NewsComment> newsComment, List<NewsLike> newsLikes) {
         this.member = member;
         this.title = title;
         this.video = video;
         this.content = content;
+        this.views = views;
         this.newsComment = newsComment;
         this.newsLikes = newsLikes;
     }
@@ -57,6 +59,7 @@ public static News create(Member member, String title, Video video, String conte
                 .title(title)
                 .video(video)
                 .content(content)
+                .views(0)
                 .newsComment(new ArrayList<>())
                 .newsLikes(new ArrayList<>())
                 .build();
@@ -80,6 +83,10 @@ public void unLike(NewsLike newsLike) {
         this.newsLikes.remove(newsLike);
     }
 
+    public void incrementViews() {
+        this.views += 1;
+    }
+
     public void addComment(NewsComment newsComment) {
         if (newsComment == null) {
             throw new IllegalArgumentException("Comment cannot be null");
diff --git a/back/src/main/java/com/back/domain/news/news/service/NewsService.java b/back/src/main/java/com/back/domain/news/news/service/NewsService.java
@@ -34,15 +34,20 @@ public News getNewsById(Long id) {
 
     public News updateNews(Member member, News news, String title, Video video, String content) {
         if (!(member.getRole() == Member.Role.ADMIN)) {
-            throw new ServiceException("403","수정 권한이 없습니다.");
+            throw new ServiceException("403", "수정 권한이 없습니다.");
         }
         news.update(title, video, content);
         return newsRepository.save(news);
     }
 
+    public News incrementViews(News news) {
+        news.incrementViews();
+        return newsRepository.save(news);
+    }
+
     public void deleteNews(Member member, News news) {
         if (!(member.getRole() == Member.Role.ADMIN)) {
-            throw new ServiceException("403","수정 권한이 없습니다.");
+            throw new ServiceException("403", "수정 권한이 없습니다.");
         }
         newsRepository.delete(news);
     }
diff --git a/back/src/main/java/com/back/global/kafka/KafkaConfig.java b/back/src/main/java/com/back/global/kafka/KafkaConfig.java
@@ -0,0 +1,38 @@
+package com.back.global.kafka;
+
+import org.apache.kafka.clients.consumer.ConsumerConfig;
+import org.apache.kafka.common.serialization.StringDeserializer;
+import org.springframework.context.annotation.Bean;
+import org.springframework.context.annotation.Configuration;
+import org.springframework.kafka.annotation.EnableKafka;
+import org.springframework.kafka.config.ConcurrentKafkaListenerContainerFactory;
+import org.springframework.kafka.core.ConsumerFactory;
+import org.springframework.kafka.core.DefaultKafkaConsumerFactory;
+
+import java.util.HashMap;
+import java.util.Map;
+
+@EnableKafka
+@Configuration
+public class KafkaConfig {
+
+    @Bean
+    public ConsumerFactory<String, String> consumerFactory() {
+        Map<String, Object> props = new HashMap<>();
+
+        props.put(ConsumerConfig.BOOTSTRAP_SERVERS_CONFIG, "localhost:9092");
+        props.put(ConsumerConfig.GROUP_ID_CONFIG, "spring-boot-group");
+        props.put(ConsumerConfig.AUTO_OFFSET_RESET_CONFIG, "earliest");
+        props.put(ConsumerConfig.KEY_DESERIALIZER_CLASS_CONFIG, StringDeserializer.class);
+        props.put(ConsumerConfig.VALUE_DESERIALIZER_CLASS_CONFIG, StringDeserializer.class);
+
+        return new DefaultKafkaConsumerFactory<>(props);
+    }
+
+    @Bean
+    public ConcurrentKafkaListenerContainerFactory<String, String> kafkaListenerContainerFactory() {
+        ConcurrentKafkaListenerContainerFactory<String, String> factory = new ConcurrentKafkaListenerContainerFactory<>();
+        factory.setConsumerFactory(consumerFactory());
+        return factory;
+    }
+}
diff --git a/back/src/main/java/com/back/global/kafka/KafkaConsumer.java b/back/src/main/java/com/back/global/kafka/KafkaConsumer.java
@@ -0,0 +1,31 @@
+package com.back.global.kafka;
+
+import com.back.domain.file.video.service.FileManager;
+import com.fasterxml.jackson.databind.JsonNode;
+import com.fasterxml.jackson.databind.ObjectMapper;
+import lombok.RequiredArgsConstructor;
+import org.springframework.kafka.annotation.KafkaListener;
+import org.springframework.stereotype.Component;
+
+@Component
+@RequiredArgsConstructor
+public class KafkaConsumer {
+    private final FileManager fileManager;
+    private final ObjectMapper mapper;
+
+    @KafkaListener(topics = "transcoding-status")
+    public void consume(String transcodingStatusMessage) {
+
+        try {
+            JsonNode rootNode = mapper.readTree(transcodingStatusMessage);
+
+            String uuid = rootNode.get("key").asText();
+            String status = rootNode.get("qualities").toString();
+            //현재 duration이 0인 문제가 있음 영상 길이가 필요한게 아니라면 제거할 예정
+            fileManager.updateVideoStatus(uuid, status);
+
+        } catch (Exception e) {
+            e.printStackTrace();
+        }
+    }
+}
diff --git a/back/src/main/java/com/back/global/kafka/KafkaProducer.java b/back/src/main/java/com/back/global/kafka/KafkaProducer.java
@@ -0,0 +1,4 @@
+package com.back.global.kafka;
+
+public class KafkaProducer {
+}
diff --git a/back/src/test/java/com/back/domain/file/entity/VideoTest.java b/back/src/test/java/com/back/domain/file/entity/VideoTest.java
@@ -15,16 +15,14 @@ void videoCreationTest() {
         String status = "{\"status\":\"done\"}";
         String originalPath = "/videos/sample.mp4";
         Integer duration = 120;
-        Long fileSize = 1024L;
 
-        Video video = Video.create(uuid, status, originalPath, duration, fileSize);
+        Video video = Video.create(uuid, status, originalPath, duration);
 
         assertThat(video).isNotNull();
         assertThat(video.getUuid()).isEqualTo(uuid);
         assertThat(video.getStatus()).isEqualTo(status);
         assertThat(video.getPath()).isEqualTo(originalPath);
         assertThat(video.getDuration()).isEqualTo(duration);
-        assertThat(video.getFileSize()).isEqualTo(fileSize);
     }
 
     @Test
@@ -34,13 +32,13 @@ void videoCreationTestWithInvalidUuid() {
         String originalPath = "/videos/sample.mp4";
 
         try {
-            Video.create(null, status, originalPath, 100, 1000L);
+            Video.create(null, status, originalPath, 100);
         } catch (Exception e) {
             assertThat(e).isInstanceOf(IllegalArgumentException.class);
         }
 
         try {
-            Video.create("", status, originalPath, 100, 1000L);
+            Video.create("", status, originalPath, 100);
         } catch (Exception e) {
             assertThat(e).isInstanceOf(IllegalArgumentException.class);
         }
@@ -53,13 +51,13 @@ void videoCreationTestWithInvalidOriginalPath() {
         String status = "{}";
 
         try {
-            Video.create(uuid, status, null, 100, 1000L);
+            Video.create(uuid, status, null, 100);
         } catch (Exception e) {
             assertThat(e).isInstanceOf(IllegalArgumentException.class);
         }
 
         try {
-            Video.create(uuid, status, "", 100, 1000L);
+            Video.create(uuid, status, "", 100);
         } catch (Exception e) {
             assertThat(e).isInstanceOf(IllegalArgumentException.class);
         }
@@ -74,7 +72,7 @@ void videoUpdateStatusTest() {
         Integer duration = 120;
         Long fileSize = 1024L;
 
-        Video video = Video.create(uuid, status, originalPath, duration, fileSize);
+        Video video = Video.create(uuid, status, originalPath, duration);
         assertThat(video.getStatus()).isEqualTo(status);
 
         String newStatus = "{\"status\":\"done\"}";
@@ -91,7 +89,7 @@ void videoUpdateStatusTestWithInvalidStatus() {
         Integer duration = 120;
         Long fileSize = 1024L;
 
-        Video video = Video.create(uuid, status, originalPath, duration, fileSize);
+        Video video = Video.create(uuid, status, originalPath, duration);
 
         try {
             video.updateStatus(null);
diff --git a/back/src/test/java/com/back/domain/file/service/VideoServiceTest.java b/back/src/test/java/com/back/domain/file/service/VideoServiceTest.java
@@ -27,7 +27,7 @@ class VideoServiceTest {
     private VideoService videoService;
 
     @Test
-    @DisplayName("transcodingResults, originalPath, originalFileName, duration, fileSize로 Video 객체 생성")
+    @DisplayName("transcodingResults, originalPath, originalFileName, duration로 Video 객체 생성")
     void videoCreationTest() {
         String uuid = UUID.randomUUID().toString();
         String transcodingResults = """
@@ -51,18 +51,16 @@ void videoCreationTest() {
                 """;
         String originalPath = "/videos/sample.mp4";
         Integer duration = 120;
-        Long fileSize = 1024L;
 
-        Video video = VideoFixture.create(uuid, transcodingResults, originalPath, duration, fileSize);
+        Video video = VideoFixture.create(uuid, transcodingResults, originalPath, duration);
         when(videoRepository.save(any(Video.class))).thenReturn(video);
 
-        Video createdVideo = videoService.createVideo(uuid, transcodingResults, originalPath, duration, fileSize);
+        Video createdVideo = videoService.createVideo(uuid, transcodingResults, originalPath, duration);
         assertThat(createdVideo).isNotNull();
         assertThat(createdVideo.getUuid()).isEqualTo(uuid);
         assertThat(createdVideo.getStatus()).isEqualTo(transcodingResults);
         assertThat(createdVideo.getPath()).isEqualTo(originalPath);
         assertThat(createdVideo.getDuration()).isEqualTo(duration);
-        assertThat(createdVideo.getFileSize()).isEqualTo(fileSize);
     }
 
     @Test
diff --git a/back/src/test/java/com/back/domain/news/news/entity/NewsTest.java b/back/src/test/java/com/back/domain/news/news/entity/NewsTest.java
@@ -291,4 +291,22 @@ void removeNullCommentTest() {
 
         assertThat(news.getNewsComment().size()).isEqualTo(0);
     }
+
+    @Test
+    @DisplayName("뉴스 조회수를 증가시킬 수 있다.")
+    void incrementViewsTest() {
+        Member member = MemberFixture.createDefault();
+        String title = "Sample News Title";
+        String content = "This is a sample news content.";
+        Video video = VideoFixture.createDefault();
+        News news = News.create(member, title, video, content);
+
+        assertThat(news.getViews()).isEqualTo(0);
+
+        news.incrementViews();
+        assertThat(news.getViews()).isEqualTo(1);
+
+        news.incrementViews();
+        assertThat(news.getViews()).isEqualTo(2);
+    }
 }
diff --git a/back/src/test/java/com/back/domain/news/news/service/NewsServiceTest.java b/back/src/test/java/com/back/domain/news/news/service/NewsServiceTest.java
@@ -175,4 +175,21 @@ void deleteNews_NoPermission_ThrowsException() {
         });
         verify(newsRepository, never()).delete(any(News.class));
     }
+
+    @Test
+    @DisplayName("뉴스 조회수 증가 성공")
+    void incrementViews_Success() {
+        // given
+        News news = NewsFixture.createDefault();
+        int initialViews = news.getViews();
+        when(newsRepository.save(any(News.class))).thenReturn(news);
+
+        // when
+        News updatedNews = newsService.incrementViews(news);
+
+        // then
+        assertThat(updatedNews).isNotNull();
+        assertThat(updatedNews.getViews()).isEqualTo(initialViews + 1);
+        verify(newsRepository, times(1)).save(news);
+    }
 }
diff --git a/back/src/test/java/com/back/fixture/NewsFixture.java b/back/src/test/java/com/back/fixture/NewsFixture.java
diff --git a/back/src/test/java/com/back/fixture/VideoFixture.java b/back/src/test/java/com/back/fixture/VideoFixture.java
diff --git a/consumer.py b/consumer.py

Original file line number	Diff line number	Diff line change
`@@ -66,6 +66,8 @@ dependencies {`
`66`	`66`	`testRuntimeOnly("org.junit.platform:junit-platform-launcher")`
`67`	`67`
`68`	`68`	`implementation ("software.amazon.awssdk:s3:2.25.0")`
	`69`	`+`
	`70`	`+ implementation ("org.springframework.kafka:spring-kafka")`
`69`	`71`	`}`
`70`	`72`
`71`	`73`	`tasks.withType<Test> {`
Original file line number	Diff line number	Diff line change
`@@ -29,4 +29,13 @@ public PresignedUrlResponse getDownloadUrl(String objectKey) {`
`29`	`29`	`LocalDateTime expiresAt = LocalDateTime.now().plusMinutes(expires);`
`30`	`30`	`return new PresignedUrlResponse(url, expiresAt);`
`31`	`31`	`}`
	`32`	`+`
	`33`	`+ //TODO : 테스트 작성필요`
	`34`	`+ public void updateVideoStatus(String videoId, String status) {`
	`35`	`+ try {`
	`36`	`+ videoService.updateStatus(videoId, status);`
	`37`	`+ } catch (Exception e) {`
	`38`	`+ videoService.createVideo(videoId, status, "/", 0);`
	`39`	`+ }`
	`40`	`+ }`
`32`	`41`	`}`
Original file line number	Diff line number	Diff line change
`@@ -11,8 +11,8 @@`
`11`	`11`	`public class VideoService {`
`12`	`12`	`private final VideoRepository videoRepository;`
`13`	`13`
`14`		`- public Video createVideo(String uuid, String status, String path, Integer duration, Long fileSize) {`
`15`		`- Video video = Video.create(uuid, status, path, duration, fileSize);`
	`14`	`+ public Video createVideo(String uuid, String status, String path, Integer duration) {`
	`15`	`+ Video video = Video.create(uuid, status, path, duration);`
`16`	`16`	`return videoRepository.save(video);`
`17`	`17`	`}`
`18`	`18`
Original file line number	Diff line number	Diff line change
`@@ -45,6 +45,7 @@ public RsData<NewsCreateResponse> createNews(@RequestBody NewsCreateRequest requ`
`45`	`45`	`@Operation(summary = "뉴스 단건 조회", description = "특정 ID의 뉴스를 읽어옵니다.")`
`46`	`46`	`public RsData<NewsGetResponse> getNews(@PathVariable("newsId") Long newsId) {`
`47`	`47`	`News news = newsService.getNewsById(newsId);`
	`48`	`+ newsService.incrementViews(news);`
`48`	`49`	`NewsGetResponse response = new NewsGetResponse(news);`
`49`	`50`	`return new RsData<>("200", "뉴스 읽어오기 완료", response);`
`50`	`51`	`}`
Original file line number	Diff line number	Diff line change
`@@ -34,15 +34,20 @@ public News getNewsById(Long id) {`
`34`	`34`
`35`	`35`	`public News updateNews(Member member, News news, String title, Video video, String content) {`
`36`	`36`	`if (!(member.getRole() == Member.Role.ADMIN)) {`
`37`		`- throw new ServiceException("403","수정 권한이 없습니다.");`
	`37`	`+ throw new ServiceException("403", "수정 권한이 없습니다.");`
`38`	`38`	`}`
`39`	`39`	`news.update(title, video, content);`
`40`	`40`	`return newsRepository.save(news);`
`41`	`41`	`}`
`42`	`42`
	`43`	`+ public News incrementViews(News news) {`
	`44`	`+ news.incrementViews();`
	`45`	`+ return newsRepository.save(news);`
	`46`	`+ }`
	`47`	`+`
`43`	`48`	`public void deleteNews(Member member, News news) {`
`44`	`49`	`if (!(member.getRole() == Member.Role.ADMIN)) {`
`45`		`- throw new ServiceException("403","수정 권한이 없습니다.");`
	`50`	`+ throw new ServiceException("403", "수정 권한이 없습니다.");`
`46`	`51`	`}`
`47`	`52`	`newsRepository.delete(news);`
`48`	`53`	`}`