Skip to content
Merged
Show file tree
Hide file tree
Changes from all commits
Commits
File filter

Filter by extension

Filter by extension

Conversations
Failed to load comments.
Loading
Jump to
Jump to file
Failed to load files.
Loading
Diff view
Diff view
Original file line number Diff line number Diff line change
Expand Up @@ -8,7 +8,7 @@
import software.amazon.awssdk.services.s3.S3Client;

@Configuration
@Profile("prod")
@Profile("prod | dev")
public class S3ClientConfig {

@Value("${cloud.aws.region.static}")
Expand Down
Original file line number Diff line number Diff line change
Expand Up @@ -11,14 +11,16 @@
import java.nio.file.Paths;
import lombok.extern.slf4j.Slf4j;
import org.springframework.beans.factory.annotation.Value;
import org.springframework.context.annotation.Profile;
import org.springframework.http.HttpStatus;
import org.springframework.stereotype.Component;

@Slf4j
@Loggable
@Component
@Profile("dev")
@Deprecated(since = "2025/11/27")
/**
* @Deprecated 날짜 : 2025/11/27
* Dev 환경 로컬 저장 방식에서 S3 방식으로 변경되어 사용하지 않게 되었습니다.
* 하지만, 추후 로컬에서 사용할 수 있는 가능성이 있어 남겨둡니다.
*/
Comment on lines -21 to +23
Copy link
Contributor

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

@Deprecated 어노테이션을 적용해도 스프링이 LocalStorageManager 컴포넌트를 빈으로 등록할 것 같습니다.
@Profile 어노테이션이 삭제되면 모든 프로필 상황에서 빈이 로드되니 S3StorageManager와 충돌이 날 것 같습니다.
아예 파일을 삭제하는 건 어떤가요?

맥에서 실행하는 local 환경에서는 MockStorageManager가 빈 등록되어 큰 문제는 없어보입니다.

Copy link
Contributor Author

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

삭제하는 방향으로 할까요? 언제 로컬에 저장할 지 몰라서 임시로 제거해둔 건데, 팀원들 의견 들어보고 결정합시다!

public class LocalStorageManager implements StorageManager {

@Value("${local.storage.base-path}")
Expand Down
Original file line number Diff line number Diff line change
Expand Up @@ -17,7 +17,7 @@

@Loggable
@Component
@Profile("prod")
@Profile("prod | dev")
public class S3StorageManager implements StorageManager {

private final S3Client s3Client;
Expand Down