Skip to content
New issue

Have a question about this project? Sign up for a free GitHub account to open an issue and contact its maintainers and the community.

By clicking “Sign up for GitHub”, you agree to our terms of service and privacy statement. We’ll occasionally send you account related emails.

Already on GitHub? Sign in to your account

chore: 프로파일 분리 및 운영환경 유틸리티 관련 기능 추가 #36

Merged
merged 9 commits into from
Jan 31, 2024

Conversation

uwoobeat
Copy link
Member

@uwoobeat uwoobeat commented Jan 31, 2024

🌱 관련 이슈

📌 작업 내용 및 특이사항

  • application.yml 을 운영환경에 맞게 분리했습니다. (local, dev, ...)
  • security, redis 설정을 분리했습니다.
  • 현재 서비스의 운영환경을 확인할 수 있는 유틸리티를 구현했습니다.
  • CORS 설정을 추가했습니다.
    • 운영환경에 따라 허용 origin을 다르게 설정하도록 구현했습니다.
  • 운영환경에 따라 쿠키의 SameSite 정책을 다르게 설정하도록 구현했습니다.
  • 그 외 자잘하게 리팩토링 필요한 부분들을 개선했습니다.

📝 참고사항

📚 기타

@uwoobeat uwoobeat self-assigned this Jan 31, 2024
@uwoobeat uwoobeat requested a review from a team as a code owner January 31, 2024 14:43
@uwoobeat uwoobeat changed the title chore: 프로파일 분리 및 스프링 운영환경 유틸리티 추가 chore: 프로파일 분리 및 운영환경 유틸리티 관련 기능 추가 Jan 31, 2024
Copy link
Member

@Sangwook02 Sangwook02 left a comment

Choose a reason for hiding this comment

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

하나만 확인 부탁드려요~

httpSecurity
.httpBasic(AbstractHttpConfigurer::disable)
public SecurityFilterChain filterChain(HttpSecurity http) throws Exception {
http.httpBasic(AbstractHttpConfigurer::disable)
.formLogin(AbstractHttpConfigurer::disable)
.csrf(AbstractHttpConfigurer::disable)
.cors(withDefaults())
Copy link
Member

Choose a reason for hiding this comment

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

이 부분은 수정 필요없나요?
.cors().configurationSource(corsConfigurationSource()) 가 되어야 하는게 아닌가요?

Copy link
Member Author

Choose a reason for hiding this comment

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

.withDefaults() 사용하더라도 커스터마이징된 설정을 로드하기 때문에 괜찮습니다.

Copy link
Member

@Sangwook02 Sangwook02 left a comment

Choose a reason for hiding this comment

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

lgtm

@uwoobeat uwoobeat merged commit b9bce54 into develop Jan 31, 2024
1 check passed
@uwoobeat uwoobeat deleted the chore/18-profile-utility branch January 31, 2024 16:05
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
None yet
Projects
None yet
Development

Successfully merging this pull request may close these issues.

⚙️ 프로파일 분리 및 스프링 운영환경 유틸리티 추가
2 participants