Skip to content

[BUG] 새 도메인 url로 변경#354

Merged
yaaan7 merged 1 commit into
developfrom
bug/353-swagger-url
Jun 15, 2026
Merged

[BUG] 새 도메인 url로 변경#354
yaaan7 merged 1 commit into
developfrom
bug/353-swagger-url

Conversation

@yaaan7

@yaaan7 yaaan7 commented Jun 15, 2026

Copy link
Copy Markdown
Collaborator

🔗 Related Issue

✨ 작업 개요

작업 내용을 간략하게 작성해주세요.

체크리스트

  • Reviewers, Assignees, Labels를 모두 등록했나요?
  • .gitignore 설정을 하였나요?
  • PR 머지 전 반드시 CI가 정상적으로 작동하는지 확인해주세요!

📷 이미지 첨부 (선택)

  • 작업 결과를 확인할 수 있는 이미지나 GIF를 첨부해주세요.
  • UI 변경, API 응답 샘플, 테스트 결과 등이 포함되면 좋아요!

🧐 집중 리뷰 요청

리뷰어가 특별히 봐주었으면 하는 부분이 있다면 작성해주세요.

@yaaan7 yaaan7 self-assigned this Jun 15, 2026
@yaaan7 yaaan7 added the 🐞 bug 버그 수정 label Jun 15, 2026

@gemini-code-assist gemini-code-assist Bot left a comment

Copy link
Copy Markdown
Contributor

Choose a reason for hiding this comment

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

Code Review

This pull request updates the production server URL in SwaggerConfig.java to use a custom domain with HTTPS. The reviewer recommended avoiding hardcoding this URL in the source code and instead injecting it dynamically from configuration files using @value to improve flexibility.

Important

The consumer version of Gemini Code Assist on GitHub is being sunset. Starting June 18, 2026, new organization installations will be blocked, and all code review activity will officially cease on July 17, 2026.
For more details on the timeline and next steps, please review the Help Documentation.


Server httpServer = new Server()
.url("http://issueissyu-backend-prod-env.eba-x2fpm3aq.ap-northeast-2.elasticbeanstalk.com")
.url("https://spring.issueissyu.cloud")

Copy link
Copy Markdown
Contributor

Choose a reason for hiding this comment

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

medium

운영 서버의 URL을 소스 코드에 직접 하드코딩하는 것은 환경 변화에 유연하게 대처하기 어렵게 만듭니다. 이 설정은 application.yml 등의 설정 파일로 분리하고, @Value 어노테이션이나 Environment를 통해 동적으로 주입받아 사용하는 것을 권장합니다.\n\n예를 들어, 다음과 같이 개선할 수 있습니다:\n\njava\n@Bean\npublic OpenAPI openAPI(@Value(\"${swagger.prod-url:https://spring.issueissyu.cloud}\") String prodUrl) {\n // ...\n Server httpServer = new Server()\n .url(prodUrl)\n .description(\"IssueIssyu Prod Server\");\n // ...\n}\n

@yaaan7 yaaan7 merged commit e8c6561 into develop Jun 15, 2026
1 check passed
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Labels

🐞 bug 버그 수정

Projects

None yet

Development

Successfully merging this pull request may close these issues.

[BUG] 스웨거 주소 변경

1 participant