-
Notifications
You must be signed in to change notification settings - Fork 3
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
HOTFIX 52 - 나눔 전체 내역 조회 조건문 변경 #52
base: main
Are you sure you want to change the base?
Conversation
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
.exposedHeaders("*") 와 시연 설정만 원복하면 될 것 같습니다! 🦾
@@ -12,6 +12,7 @@ class CorsConfig : WebMvcConfigurer { | |||
.allowedOriginPatterns("*") // 모든 도메인에 대해 액세스 허용 | |||
.allowedMethods("GET", "POST", "PUT", "PATCH", "DELETE", "OPTIONS") // 허용할 HTTP 메서드 | |||
.allowedHeaders("*") // 모든 헤더 허용 | |||
.exposedHeaders("*") |
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
실제 header 값인 access token 과 refresh token , re-access token 등으로 변경하면 좋을 것 같습니다! 😄
Reference
@@ -35,7 +35,7 @@ cloud: | |||
|
|||
jwt: | |||
secret-key: ${jwt-secret-key} | |||
access-duration-mils: 1800000 | |||
access-duration-mils: 21600000 |
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
시연용 설정은 원복해서 다시 배포하는 것이 좋을 것 같습니다! 🎡
Pull Request
작업 내용
이 PR은 다음과 같은 작업을 포함하고 있습니다:
변경 사항
다음과 같은 파일/부분을 수정했습니다:
src/main/kotlin/mara/server/domain/�share/ShareRepository.kt
: findAllMyShare 쿼리문 조건 and 에서 or 로 변경테스트
다음과 같은 테스트를 수행했습니다: