[한성재] 스프린트 미션8 by seonghj · Pull Request #191 · codeit-bootcamp-spring/9-sprint-mission

seonghj · 2026-04-19T01:03:07Z

요구사항

기본

애플리케이션 컨테이너화
BinaryContentStorage 고도화
AWS를 활용한 배포

심화

이미지 최적화하기
GitHub Actions를 활용한 CI/CD 파이프라인 구축

스크린샷

RDS

ECR

ECS

보안 그룹

IAM 권한 정책

.env 파일

# Spring Configuration
SPRING_PROFILES_ACTIVE=prod
PORT=80

# Application Configuration
STORAGE_TYPE=s3
AWS_S3_ACCESS_KEY=
AWS_S3_SECRET_KEY=
AWS_S3_REGION=ap-northeast-2
AWS_S3_BUCKET=discodeit-binary-content-storage-hsj
AWS_S3_PRESIGNED_URL_EXPIRATION=600

# DataSource Configuration
RDS_ENDPOINT=discodeit-db.cheuy484ure1.ap-northeast-2.rds.amazonaws.com:5432
SPRING_DATASOURCE_URL=jdbc:postgresql://${RDS_ENDPOINT}/discodeit
SPRING_DATASOURCE_USERNAME=discodeit_user
SPRING_DATASOURCE_PASSWORD=discodeit1234

# JVM Configuration
JVM_OPTS=-Xmx256m -Xms256m -XX:MaxMetaspaceSize=192m -XX:+UseSerialGC

멘토에게

배포된 EC2 엔드포인트 = http://3.39.248.137/

기본 요구사함 프로파일 기반 설정 관리에 맞춘 프로파일 추가

Spring Validation 의존성을 추가

주요 Request에 제약 조건 관련 어노테이션 추가, GlobalExceptionHandler에 handleMethodArgumentNotValidException 추가

심화 과정 테스트

심화과정 테스트 2

심화 테스트 3

심화 요구사항 테스트 4

Update deploy.yml

This reverts commit 9e4a02a.

This reverts commit d399fa6.

.

피드백 반영

한성재 sprint8

joonfluence

전체 요약

Docker 컨테이너화, AWS S3 BinaryContent 고도화, ECS/ECR 기반 배포, GitHub Actions CI/CD 파이프라인을 구축한 PR입니다. 전반적인 구조는 잘 설계되어 있으나, IAM Role 환경에서 presigned URL 생성 불일치, 사용자 삭제 시 NPE, 배포 중 다운타임 등 수정이 필요한 이슈가 있습니다.

[P4] 커밋에 포함된 불필요한 파일

logs/discodeit.log (768줄) — 런타임 로그 파일이 커밋되었습니다. .gitignore에 logs/가 추가되었으므로 git rm --cached logs/discodeit.log로 추적에서 제외해 주세요.
discodeit/.gradle/buildOutputCleanup/ — Gradle 빌드 바이너리 파일이 커밋되었습니다. .gitignore에 추가하고 제거가 필요합니다.

⚠️ 보안 참고: PR 설명에 실제 RDS 비밀번호(discodeit1234)와 EC2 퍼블릭 IP가 노출되어 있습니다. 해당 자격증명이 실제 운영 환경에서 사용 중이라면 즉시 변경을 권장합니다.

joonfluence · 2026-04-21T15:59:05Z

+  private String generatePresignedUrl(String key, String contentType) {
+    try (S3Presigner presigner = S3Presigner.builder()
+        .region(Region.of(region))
+        .credentialsProvider(StaticCredentialsProvider.create(


[P1] IAM Role 환경에서 presigned URL 생성 실패 위험

generatePresignedUrl()에서 S3Presigner가 항상 StaticCredentialsProvider(하드코딩 자격증명)를 사용합니다. 그러나 S3Config는 access key가 비어있으면 DefaultCredentialsProvider(IAM Role 등)를 사용하도록 설계되어 있습니다.

ECS에서 IAM Role로 실행 시 awsProperties.getAccessKey()가 빈 값이 되어 presigned URL 생성이 실패합니다. S3Config와 동일한 자격증명 결정 로직을 적용해야 합니다.

Suggested change

.credentialsProvider(StaticCredentialsProvider.create(

private String generatePresignedUrl(String key, String contentType) {

software.amazon.awssdk.auth.credentials.AwsCredentialsProvider credProvider;

if (accessKey != null && !accessKey.isBlank()) {

credProvider = StaticCredentialsProvider.create(

AwsBasicCredentials.create(accessKey, secretKey));

} else {

credProvider = software.amazon.awssdk.auth.credentials.DefaultCredentialsProvider.create();

}

try (S3Presigner presigner = S3Presigner.builder()

.region(Region.of(region))

.credentialsProvider(credProvider)

.build()) {

joonfluence · 2026-04-21T15:59:05Z

+  private final S3Client s3Client;
+  private final AwsProperties awsProperties;
+
+  private String accessKey;


[P3] AwsProperties 중복 복사 및 미사용 메서드

AwsProperties가 이미 주입되어 있는데, accessKey, secretKey, region, bucket을 별도 인스턴스 필드로 복사하고 있습니다. 또한 68번 줄의 getS3Client() 메서드도 실제로 사용되지 않습니다.

awsProperties.getXxx()를 직접 참조하거나, 불필요한 필드와 메서드를 제거하면 코드가 단순해집니다.

joonfluence · 2026-04-21T15:59:06Z

-            binaryContentRepository.deleteById(removeUser.getProfile().getId());
-            userRepository.deleteById(removeUser.getId());
+          userStatusRepository.deleteById(userStatus.getId());
+          binaryContentRepository.deleteById(removeUser.getProfile().getId());


[P1] 프로필 없는 사용자 삭제 시 NullPointerException

프로필 이미지가 없는 사용자(profile == null)를 삭제하면 removeUser.getProfile().getId()에서 NPE가 발생합니다. 프로필은 선택적(optional)이므로 null 체크가 필요합니다.

Suggested change

binaryContentRepository.deleteById(removeUser.getProfile().getId());

if (removeUser.getProfile() != null) {

binaryContentRepository.deleteById(removeUser.getProfile().getId());

}

joonfluence · 2026-04-21T15:59:06Z

+          container-name: discodeit-app
+          image: ${{ vars.ECR_REPOSITORY_URI }}:latest
+
+      - name: Stop and Wait for Task to Terminate


[P2] 배포 중 다운타임 발생

desired-count 0으로 서비스를 완전히 중단한 뒤 새 태스크를 배포하므로, "중단 → 배포 → Restart" 사이 구간에 다운타임이 발생합니다.

Stop and Wait 및 Restart Service 스텝을 제거하고 amazon-ecs-deploy-task-definition 액션만으로 롤링 업데이트를 처리하면 무중단 배포가 가능합니다. 단일 태스크 환경이라면 ECS 서비스의 minimumHealthyPercent: 0, maximumPercent: 100 설정으로 제어할 수 있습니다.

joonfluence · 2026-04-21T15:59:06Z

+          task-definition: ${{ steps.task-def.outputs.task-definition }}
+          service: ${{ vars.ECS_SERVICE }}
+          cluster: ${{ vars.ECS_CLUSTER }}
+          wait-for-service-stability: false


[P3] 배포 성공 여부 미확인

배포 후 ECS 서비스 안정화를 기다리지 않으므로, 새 태스크 시작이 실패해도 파이프라인이 성공으로 표시됩니다. true로 변경하면 배포 실패를 즉시 감지할 수 있습니다.

Suggested change

wait-for-service-stability: false

wait-for-service-stability: true

joonfluence · 2026-04-21T15:59:06Z

-        ApiError error = new ApiError("SERVER ERROR", "IllegalArgumentException - " + e.getMessage());
-        return ResponseEntity.status(HttpStatus.INTERNAL_SERVER_ERROR).body(error);
+    @ExceptionHandler(Exception.class)
+    public ResponseEntity<ErrorResponse> handleException(Exception e) {


[P3] catch-all 핸들러에서 예외 미로깅

handleException(Exception e) catch-all 핸들러에서 log.error()가 없어, 예상치 못한 예외 발생 시 스택 트레이스가 로깅되지 않습니다. 운영 환경에서 디버깅이 매우 어려워질 수 있습니다.

Suggested change

public ResponseEntity<ErrorResponse> handleException(Exception e) {

public ResponseEntity<ErrorResponse> handleException(Exception e) {

log.error("처리되지 않은 예외 발생", e);

joonfluence · 2026-04-21T15:59:06Z

+            .body(response);
+    }
+    // User
+    @ExceptionHandler(UserNotFoundException.class)


[P3] DiscodeitException 하위 클래스 핸들러 중복

UserNotFoundException, UserAlreadyExistsException, ChannelNotFoundException, PrivateChannelUpdateException, MessageNotFoundException 핸들러가 모두 83번 줄의 handleDiscodeitException과 동일한 로직(ErrorResponse.of(e, status.value()))을 수행합니다.

Spring은 @ExceptionHandler에서 가장 구체적인 타입을 우선 적용하지만, 이 하위 클래스들은 이미 DiscodeitException 핸들러로 처리됩니다. 중복 핸들러를 제거하면 새로운 예외 클래스 추가 시 일관성을 유지하기 쉬워집니다.

seonghj and others added 30 commits March 25, 2026 13:38

dev + prod.yaml 생성

bb00da3

기본 요구사함 프로파일 기반 설정 관리에 맞춘 프로파일 추가

로그 적용 + 커스텀 예외 생성

99cf940

서비스에 커스텀 예외 적용

2e4dd0b

GlobalExceptionHandler 수정

a1e0ea9

build.gradle 수정

8f652c7

Spring Validation 의존성을 추가

유효성 검사 구현

0666e59

주요 Request에 제약 조건 관련 어노테이션 추가, GlobalExceptionHandler에 handleMethodArgumentNotValidException 추가

Spring Boot Actuator 추가

c461643

UserService 단위 테스트

dc3e32f

단위 테스트 BDDMockito 적용

60dfe8a

통합 테스트 추가

cf2ffc2

심화 구현 요소 추가

497032a

controller 로그 추가

5fdcb29

BinaryContentStorage 고도화

50e1c1d

s3BinaryContentStorageTest

ae45811

Update Dockerfile

c238858

Create test.yml

f95e257

Update build.gradle

42d7813

Update test.yml

3186279

test.yml working-directory 추가

ed3dff8

Update test.yml

b531bed

Update test.yml

bd60909

Update test.yml

7e8e537

build.gradle 수정

6ac825b

AwsProperties 수정

c2a59a4

..

3e566d3

Update test.yml

09633ca

Create README.md

2190a15

Create deploy.yml

dbecb6d

Update deploy.yml

cc38a36

Update deploy.yml

ecffc5d

seonghj and others added 28 commits April 10, 2026 14:29

Update deploy.yml

099995c

Update deploy.yml

e17a30b

Update deploy.yml

455a043

Update deploy.yml

7ca95c3

Update deploy.yml

803da91

Update deploy.yml

ade9102

Update deploy.yml

bdb31bd

Merge pull request #4 from seonghj/한성재_sprint8

c2602b4

심화 과정 테스트

Update deploy.yml

e46754c

Merge pull request #5 from seonghj/한성재_sprint8

8be9921

심화과정 테스트 2

Update deploy.yml

14685eb

Update deploy.yml

12ade6a

Merge pull request #7 from seonghj/한성재_sprint8

8c0c465

심화 테스트 3

deploy.yml 수정

658dba1

Merge pull request #8 from seonghj/한성재_sprint8

932111d

심화 요구사항 테스트 4

Update deploy.yml

d399fa6

Merge pull request #9 from seonghj/한성재_sprint8

17726a2

Update deploy.yml

Update deploy.yml

9e4a02a

Merge pull request #10 from seonghj/한성재_sprint8

ced536f

Update deploy.yml

Revert "Update deploy.yml"

a366b48

This reverts commit 9e4a02a.

Revert "Update deploy.yml"

854733c

This reverts commit d399fa6.

Merge pull request #11 from seonghj/한성재_sprint8

c5d8320

.

피드백 반영

5900baf

findAllByChannel_Id_Success 수정

01c113f

Merge pull request #12 from seonghj/한성재_sprint8

e4b403b

피드백 반영

Update README.md

795490f

Update README.md

f58d53e

Merge pull request #13 from seonghj/한성재_sprint8

023fc83

한성재 sprint8

joonfluence suggested changes Apr 21, 2026

View reviewed changes

Provide feedback

Saved searches

Use saved searches to filter your results more quickly

[한성재] 스프린트 미션8#191

Hidden character warning

[한성재] 스프린트 미션8#191
seonghj wants to merge 58 commits into
codeit-bootcamp-spring:한성재from
seonghj:한성재_release

seonghj commented Apr 19, 2026

Uh oh!

joonfluence left a comment

Uh oh!

joonfluence Apr 21, 2026

Uh oh!

joonfluence Apr 21, 2026

Uh oh!

joonfluence Apr 21, 2026

Uh oh!

joonfluence Apr 21, 2026

Uh oh!

joonfluence Apr 21, 2026

Uh oh!

joonfluence Apr 21, 2026

Uh oh!

joonfluence Apr 21, 2026

Uh oh!

Reviewers

Assignees

Labels

Projects

Milestone

Development

Uh oh!

2 participants

-        .credentialsProvider(StaticCredentialsProvider.create(
+  private String generatePresignedUrl(String key, String contentType) {
+    software.amazon.awssdk.auth.credentials.AwsCredentialsProvider credProvider;
+    if (accessKey != null && !accessKey.isBlank()) {
+      credProvider = StaticCredentialsProvider.create(
+          AwsBasicCredentials.create(accessKey, secretKey));
+    } else {
+      credProvider = software.amazon.awssdk.auth.credentials.DefaultCredentialsProvider.create();
+    }
+    try (S3Presigner presigner = S3Presigner.builder()
+        .region(Region.of(region))
+        .credentialsProvider(credProvider)
+        .build()) {

	wait-for-service-stability: false
	wait-for-service-stability: true

	public ResponseEntity<ErrorResponse> handleException(Exception e) {
	public ResponseEntity<ErrorResponse> handleException(Exception e) {
	log.error("처리되지 않은 예외 발생", e);

Conversation

seonghj commented Apr 19, 2026

요구사항

기본

심화

스크린샷

멘토에게

Uh oh!

joonfluence left a comment

Choose a reason for hiding this comment

전체 요약

[P4] 커밋에 포함된 불필요한 파일

Uh oh!

joonfluence Apr 21, 2026

Choose a reason for hiding this comment

[P1] IAM Role 환경에서 presigned URL 생성 실패 위험

Uh oh!

joonfluence Apr 21, 2026

Choose a reason for hiding this comment

[P3] AwsProperties 중복 복사 및 미사용 메서드

Uh oh!

joonfluence Apr 21, 2026

Choose a reason for hiding this comment

[P1] 프로필 없는 사용자 삭제 시 NullPointerException

Uh oh!

joonfluence Apr 21, 2026

Choose a reason for hiding this comment

[P2] 배포 중 다운타임 발생

Uh oh!

joonfluence Apr 21, 2026

Choose a reason for hiding this comment

[P3] 배포 성공 여부 미확인

Uh oh!

joonfluence Apr 21, 2026

Choose a reason for hiding this comment

[P3] catch-all 핸들러에서 예외 미로깅

Uh oh!

joonfluence Apr 21, 2026

Choose a reason for hiding this comment

[P3] DiscodeitException 하위 클래스 핸들러 중복

Uh oh!

Reviewers

Assignees

Labels

Projects

Milestone

Development

Uh oh!

2 participants