feat: Parameter Store 기반 설정 관리 전환 및 loadtest 프로필 추가#670
feat: Parameter Store 기반 설정 관리 전환 및 loadtest 프로필 추가#670lsy1307 wants to merge 5 commits intosolid-connection:developfrom
Conversation
- secret 서브모듈 제거 및 설정 파일 분리 - 프로필별 Parameter Store import와 loadtest 프로필 추가
- Spring Cloud AWS Parameter Store 의존성 추가 - config 분리와 profile별 import/loadtest 프로필 설정
- dev/prod workflow에서 submodule checkout 제거 - compose 환경변수에 AWS_REGION 주입
- 정적 access key/secret key 주입 제거 - DefaultCredentialsProvider 기반 S3Client 구성
lsy1307
requested review from
Hexeong,
JAEHEE25,
sukangpunch,
whqtker and
wibaek
as code owners
Walkthrough1. Git 서브모듈 제거 —\t Estimated code review effort🎯 3 (Moderate) | ⏱️ ~25 minutes Suggested reviewers
🚥 Pre-merge checks | ✅ 5 | ❌ 1❌ Failed checks (1 warning)
✅ Passed checks (5 passed)
✏️ Tip: You can configure your own custom pre-merge checks in the settings. ✨ Finishing touches🧪 Generate unit tests (beta)
Thanks for using CodeRabbit! It's free for OSS, and your support helps us grow. If you like it, consider giving us a shout-out. Comment |
![coderabbitai[bot]](https://avatars.githubusercontent.com/in/347564?s=60&v=4){kind=small}
There was a problem hiding this comment.
Actionable comments posted: 2
🤖 Fix all issues with AI agents
In `@src/main/resources/application-loadtest.yml`:
- Around line 17-25: The outbound websocket thread-pool block is missing a
queue-capacity setting which defaults to an unbounded queue; update the
websocket.thread-pool.outbound configuration to include queue-capacity (e.g.,
match inbound's 4000) so outbound uses a bounded queue like inbound:
core-pool-size, max-pool-size, and queue-capacity should all be specified for
websocket.thread-pool.outbound to avoid unbounded memory growth during load
tests.
In `@src/main/resources/config/application-variable.yml`:
- Around line 189-191: 로컬 프로필의 token.refresh.cookie-domain이
".stage.solid-connection.com"으로 잘못 설정되어 있어 로컬에서 쿠키가 동작하지 않습니다;
token.refresh.cookie-domain 값을 로컬 전용 설정에서 "localhost" (또는 빈값/생략하여 기본 호스트 사용)로
변경하거나 로컬 프로파일(application-local.yml 등)에 별도 항목을 추가해 ".stage.solid-connection.com"
대신 localhost를 사용하도록 수정하세요.
🧹 Nitpick comments (3)
src/main/resources/config/application-variable.yml (1)
30-30: 카카오 OAuth 프로퍼티의 네이밍 일관성을 맞춰주세요.현재
user-info_url은 언더스코어와 하이픈이 혼용되어 있는데, 같은oauth섹션의 모든 다른 프로퍼티들(token-url,redirect-url,client-id,client-secret-audience-url등)은 케밥 케이스(hyphen)를 사용하고 있습니다. Spring Boot의 relaxed binding 때문에 현재도 정상 작동하지만, 일관성을 위해user-info-url로 통일하면 좋겠습니다.변경 사항:
user-info_url→user-info-url로 변경
src/main/resources/config/application-variable.yml(line 30)src/test/resources/application.yml♻️ 변경 예시
- user-info_url: "https://kapi.kakao.com/v2/user/me" + user-info-url: "https://kapi.kakao.com/v2/user/me"src/main/resources/config/application-cloud.yml (1)
6-11: Spring Cloud AWS 3.x에 맞춰 프로퍼티 네임스페이스를 정렬하세요.현재 설정에서
cloud.aws.*네임스페이스는 Spring Cloud AWS 2.x 관례입니다. 3.0.4 버전에서는 공식 네임스페이스가spring.cloud.aws.*로 변경되었습니다.다음 개선사항을 고려해 주세요:
cloud.aws.region.static→spring.cloud.aws.region.static로 변경
- AmazonS3Config에서
@Value("${spring.cloud.aws.region.static}")로 읽도록 수정- Spring Cloud AWS 자동 설정의 중앙 리전 제공자와 일관성 유지
cloud.aws.stack.auto및 기타cloud.aws.*속성도spring.cloud.aws.*로 통일
- Spring Cloud AWS 3.x의 공식 설정 구조와 맞춤
- 향후 유지보수 시 혼동 방지
현재는 Parameter Store 클라이언트가 Docker Compose의
AWS_REGION환경 변수를 통해 리전을 올바르게 해석하므로 기능상 문제가 없습니다. 다만 네임스페이스 정렬은 코드 일관성과 명확성을 크게 개선합니다.src/main/resources/application-loadtest.yml (1)
12-15: HikariCP 풀 사이즈 설정 관련 참고 사항
- 풀 사이즈
maximum-pool-size: 50,minimum-idle: 20은 부하테스트 용도로 합리적인 수치입니다.- 타임아웃 설정 부재
connection-timeout,idle-timeout등이 명시되어 있지 않아 HikariCP 기본값(각각 30초, 600초)이 적용됩니다.- K6 부하테스트 시 커넥션 풀 소진 상황에서 기본
connection-timeout(30초)이 적절한지 확인해 보시면 좋겠습니다. 필요하다면 부하테스트 특성에 맞게 조정을 고려해 주세요.
- local 프로필 refresh cookie-domain을 localhost로 수정 - loadtest outbound thread pool에 queue-capacity 추가
There was a problem hiding this comment.
Actionable comments posted: 2
🤖 Fix all issues with AI agents
In `@src/main/resources/config/application-variable.yml`:
- Around line 134-142: The dev CORS list under cors.allowed-origins contains the
prod admin domain; update the entry "https://admin.solid-connection.com" in the
dev profile to the staging admin domain
"https://admin.stage.solid-connection.com" (or remove/replace it) so dev uses
the stage admin host; if the admin service intentionally uses a single global
domain, leave it as-is but add a comment explaining that choice.
- Around line 12-14: Add an explicit queue-capacity entry for the outbound
thread pool in the default and prod profiles: update the outbound block (keys:
outbound, core-pool-size, max-pool-size) to include queue-capacity (e.g., 1000
or another bounded value) so Spring's ThreadPoolTaskExecutor does not use
Integer.MAX_VALUE; ensure the same queue-capacity is added under both the
default outbound section and the prod outbound section to mirror inbound's
configuration.
🧹 Nitpick comments (2)
src/main/resources/config/application-variable.yml (2)
176-177: dev/local 프로필에서JWT_SECRET의 기본값이 비어 있습니다.
${JWT_SECRET:}는 환경 변수가 설정되지 않으면 빈 문자열로 바인딩됩니다. 로컬 개발 시 Parameter Store 없이 실행하면 JWT 서명이 빈 키로 수행될 수 있어, 토큰 검증이 의미 없어지거나 예기치 않은 오류가 발생할 수 있습니다. 로컬용으로는 안전한 기본 개발용 시크릿을 넣어두는 것을 권장합니다.🛡️ 로컬 프로필에 개발용 기본 시크릿 추가 제안
jwt: - secret: ${JWT_SECRET:} + secret: ${JWT_SECRET:local-dev-only-secret-do-not-use-in-production}
30-30:user-info_url키의 네이밍 불일치를 수정하세요.파일 전체에서 kebab-case를 일관되게 사용하고 있는데(token-url, client-secret-audience-url, public-key-url 등), 이 키만
user-info_url로 언더스코어가 섞여 있습니다. Spring Boot의 relaxed binding이 자동으로 처리하므로 동작에는 문제가 없지만, 코드 스타일 통일을 위해 다음을 진행해 주세요:
- 키 이름 수정:
user-info_url→user-info-url로 변경
- Java의
KakaoOAuthClientProperties.userInfoUrl바인딩은 Spring의 relaxed binding을 통해 자동으로 매핑되므로 추가 수정이 필요 없습니다.✏️ 제안된 변경
- user-info_url: "https://kapi.kakao.com/v2/user/me" + user-info-url: "https://kapi.kakao.com/v2/user/me"
| outbound: | ||
| core-pool-size: 6 | ||
| max-pool-size: 12 |
There was a problem hiding this comment.
기본(default) 프로필의 outbound 스레드 풀에 queue-capacity가 누락되어 있습니다.
inbound 쪽(Line 11)에는 queue-capacity: 1000이 명시되어 있는 반면, outbound에는 빠져 있습니다. prod 프로필(Line 75-77)에서도 동일하게 outbound queue-capacity가 누락되어 있네요. 의도적인 생략인지, 아니면 설정 누락인지 확인이 필요합니다. 만약 Spring의 ThreadPoolTaskExecutor 기본값(Integer.MAX_VALUE)이 적용된다면 사실상 무제한 큐가 되어 메모리 이슈로 이어질 수 있습니다.
🔧 outbound에도 queue-capacity 추가 제안
outbound:
core-pool-size: 6
max-pool-size: 12
+ queue-capacity: 1000📝 Committable suggestion
‼️ IMPORTANT
Carefully review the code before committing. Ensure that it accurately replaces the highlighted code, contains no missing lines, and has no issues with indentation. Thoroughly test & benchmark the code to ensure it meets the requirements.
| outbound: | |
| core-pool-size: 6 | |
| max-pool-size: 12 | |
| outbound: | |
| core-pool-size: 6 | |
| max-pool-size: 12 | |
| queue-capacity: 1000 |
🤖 Prompt for AI Agents
In `@src/main/resources/config/application-variable.yml` around lines 12 - 14, Add
an explicit queue-capacity entry for the outbound thread pool in the default and
prod profiles: update the outbound block (keys: outbound, core-pool-size,
max-pool-size) to include queue-capacity (e.g., 1000 or another bounded value)
so Spring's ThreadPoolTaskExecutor does not use Integer.MAX_VALUE; ensure the
same queue-capacity is added under both the default outbound section and the
prod outbound section to mirror inbound's configuration.
| cors: | ||
| allowed-origins: | ||
| - "https://www.stage.solid-connection.com" | ||
| - "https://api.stage.solid-connection.com" | ||
| - "https://admin.solid-connection.com" | ||
| - "http://localhost:3000" | ||
| - "https://localhost:3000" | ||
| - "http://localhost:5173" | ||
|
|
There was a problem hiding this comment.
Dev 프로필 CORS에 운영(prod) admin 도메인이 포함되어 있습니다.
Line 138의 https://admin.solid-connection.com은 prod 프로필(Line 96)에서도 동일하게 사용되는 운영 도메인입니다. dev 환경이라면 https://admin.stage.solid-connection.com이 의도된 값이 아닌지 확인 부탁드립니다. 만약 admin 서비스가 환경 구분 없이 하나의 도메인을 사용하는 구조라면 현재 설정이 맞습니다.
🤖 Prompt for AI Agents
In `@src/main/resources/config/application-variable.yml` around lines 134 - 142,
The dev CORS list under cors.allowed-origins contains the prod admin domain;
update the entry "https://admin.solid-connection.com" in the dev profile to the
staging admin domain "https://admin.stage.solid-connection.com" (or
remove/replace it) so dev uses the stage admin host; if the admin service
intentionally uses a single global domain, leave it as-is but add a comment
explaining that choice.
1 participant
관련 이슈
작업 내용
config/*.yml+ AWS Parameter Store import 방식으로 전환loadtest프로필 추가 (application-loadtest.yml)AWS_REGION=ap-northeast-2추가DefaultCredentialsProvider기반(IAM Role 체인)으로 변경특이 사항
SolidConnectionParameterStoreReadProfile연결 완료/solid-connection/{common|local|dev|prod|loadtest}사용리뷰 요구사항 (선택)
spring.datasource.*,spring.flyway.*,jwt.secret)이 현재 운영 정책과 맞는지flyway적용 정책(활성/비활성) 확정 필요