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.

Sign up for GitHub

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

Jump to bottom

[FIX] 추가 변경 사항 반영 #90

Merged
merged 2 commits into from
Aug 12, 2023
Merged

[FIX] 추가 변경 사항 반영 #90

merged 2 commits into from
Aug 12, 2023

Conversation

jun02160
Copy link
Member

📌 관련 이슈

✨ 어떤 이유로 변경된 내용인지

  • em.close() 호출 시점을 일단 머지와 커밋 이후 아무데나 넣어뒀는데, 이렇게 하면 트랜잭션이 롤백된 이후에 close가 보장되지 않아서 finally 구문으로 바꿔뒀습니다!
  • ControllerAdvice 500에러 핸들링 주석 처리했던 부분 다시 살려뒀습니다
  • PostMan Test의 편의를 위해서 소셜 로그인 시 액세스 토큰 로그로 찍어두도록 추가했습니다!

🙏 검토 혹은 리뷰어에게 남기고 싶은 말

모듈 분리한 버전에도 동일한 내용 반영함요 ~!!

@jun02160 jun02160 requested a review from ddongseop August 12, 2023 13:38
@jun02160 jun02160 self-assigned this Aug 12, 2023
@jun02160 jun02160 added 🔥 Pull Request PR 날림 예준🍒 🐞BugFix 에러 수정, 버그 수정 🧹Chore 동작에 영향 없는 코드 or 파일, 폴더 관련 수정사항 labels Aug 12, 2023
@jun02160 jun02160 merged commit 7a2b2bd into develop Aug 12, 2023
@ddongseop ddongseop deleted the fix/#88-jpa_connection_error branch August 12, 2023 16:36
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Reviewers

@ddongseop ddongseop ddongseop approved these changes

Assignees

@jun02160 jun02160

Labels
예준🍒 🐞BugFix 에러 수정, 버그 수정 🧹Chore 동작에 영향 없는 코드 or 파일, 폴더 관련 수정사항 🔥 Pull Request PR 날림
Projects
None yet
Milestone
No milestone
Development

Successfully merging this pull request may close these issues.
2 participants
@jun02160 @ddongseop