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

9주차 미션 / 서버 4조 지호준 #6

Open
wants to merge 5 commits into
base: master
Choose a base branch
from

Conversation

ho0010
Copy link

@ho0010 ho0010 commented May 31, 2024

  1. JWT
    각 user가 자신의 정보만 수정할 수 있도록 구현하는 과정에서 처음에는 httprequest에서 userId를 가져오는 방식으로 구현했었는데 뭔가 제공해주신 코드를 잘 활용하지 못하고 있다는 느낌이 들었습니다. 오늘 스터디에서 말씀해주신 내용을 바탕으로 @PreAuthorize와 argumentresolver의 역할을 이해해 미션의도에 부합하도록 수정했습니다. 모든 메소드에 적용하지는 못했습니다.

  2. 페이징처리
    no offset 방식과 offset 방식을 강의 들을때는 잘 이해했다고 생각했으나 막상 구현하려고 하니 두 방식을 구분하는 것이 어려웠습니다. 다만 스터디원분이 이와 관련한 내용을 잘 설명해주셔서 시스템에서 쿼리문을 해석하는 방식으로 인해 no offset방식과 offset 방식이 결과적으로 구분되는 것을 알게되었고 제가 구현한 방식이 offset 방식인것을 알게되었습니다.

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.

1 participant