Skip to content

[9주차/제이] 워크북 제출합니다.#62

Open
yeongjun25 wants to merge 4 commits into
UMC-Inha:j/mainfrom
yeongjun25:week9
Open

[9주차/제이] 워크북 제출합니다.#62
yeongjun25 wants to merge 4 commits into
UMC-Inha:j/mainfrom
yeongjun25:week9

Conversation

@yeongjun25
Copy link
Copy Markdown

@yeongjun25 yeongjun25 commented May 28, 2026

✅ 실습 체크리스트

  • 이론 학습을 완료하셨나요?
  • 미션 요구사항을 모두 이해하셨나요?
  • 실습을 수행하기 위한 공부를 완료하셨나요?
  • 실습 요구사항을 모두 완료하셨나요?

✅ 컨벤션 체크리스트

  • 디렉토리 구조 컨벤션을 잘 지켰나요?
  • pr 제목을 컨벤션에 맞게 작성하였나요?
  • pr에 해당되는 이슈를 연결하였나요?(중요)
  • 적절한 라벨을 설정하였나요?
  • 파트장에게 code review를 요청하기 위해 reviewer를 등록하였나요?
  • 닉네임/main 브랜치의 최신 상태를 반영하고 있는지 확인했나요?(매우 중요!)

📌 주안점

@yeongjun25 yeongjun25 requested a review from YoungJJun May 28, 2026 14:43
@yeongjun25 yeongjun25 self-assigned this May 28, 2026
@yeongjun25 yeongjun25 linked an issue May 28, 2026 that may be closed by this pull request
Copy link
Copy Markdown
Collaborator

@YoungJJun YoungJJun left a comment

Choose a reason for hiding this comment

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

9주차 피드백

  1. 미션 정말 깔끔하게 잘 구현해주셨습니다!

  2. JwtAuthFilter

    catch 블록에서 new ObjectMapper() 생성하는 코드가 있습니다.

    private static final 로 선언해서 매번 생성하지 않도록 하는게 좋을 것 같습니다.

    혹은 이미 JacksonConfig에서 빈으로 등록했지 때문에 생성자로 주입해주는 방법도 있습니다!

  3. MemberService - login(), getMyPage()

    모두 읽기전용인 것 같습니다. @TransactionalreadOnly = true 붙이면 좋을 것 같아요.


제이 이번주 바쁘셨을텐데 미션을 너무 꼼꼼히 구현해주셔서 크게 문제되는 부분은 없는것 같아요! 덕분에 리뷰가 빨리 끝나겠어요 ㅎㅎ
시험 잘보세용~ 🍎

Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Projects

None yet

Development

Successfully merging this pull request may close these issues.

Chapter09_Spring Security - JWT, OAuth

2 participants