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

#518 [feat] 인가체제 변경 및 인가 API 중 필드 수정 #519

Merged
merged 10 commits into from
Oct 1, 2024

Conversation

sohyundoh
Copy link
Member

✒️ 관련 이슈번호

Key Changes 🔑

  1. 한 API에 의존하여 작업하기에는 다른 API가 작동하지 않을 것 같아 변경하면서 함께 변경해야하는 API를 같이 수정했습니다.
  2. 그리고, moimId를 담아보내지 않는 요청에 대해서는 Map에서 꺼내서 넘겨줄 수 없기 때문에 Map 자체를 ThreadLocal에 저장하고, 필요할 경우 MoimWriterNameMapUtil에서 꺼내쓰는 방법으로 진행했습니다! 이 방법을 착안하여 다른 API에서도 활용하시면 좋을 것 같습니다! (혹시 다른 좋은 생각이 있다면 조언 부탁드립니다!!!) 이 Util에서 꺼내쓰는 위치는 Controller입니다!
  3. 작업한 API는 아래와 같습니다! 아래를 제외하고 진행해주시면 됩니다!
  • [필명 & 소개글 GET]
  • [댓글 GET, 댓글 POST]
  • [궁금해요 POST]
  • [궁금해요 여부 및 개수 GET]
  • [궁금해요 DELETE]
  • [임시저장글 GET, 임시저장글 DELETE]
  • [글 수정을 위한 GET, 글 PUT]

To Reviewers 📢

  • 검토 후 수정하고픈 부분있으면 부탁드립니다!
  • 이런 방식으로 진행하면 될 것 같습니다!

@sohyundoh sohyundoh added the high label Sep 28, 2024
@sohyundoh sohyundoh self-assigned this Sep 28, 2024
@sohyundoh sohyundoh changed the title #518 [feat] 인가체제 변경 및 인가 API.중 필드 수정 #518 [feat] 인가체제 변경 및 인가 API 중 필드 수정 Sep 28, 2024
Copy link
Contributor

@parkheeddong parkheeddong left a comment

Choose a reason for hiding this comment

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

저도 ThreadLocal로 작성자 정보 저장하고 필요시 사용하는 방식이 좋은 것 같습니다. 확실히 중앙화되고 성능에도 좋을 것 같아요! LGTM!

@sohyundoh sohyundoh merged commit 6b35522 into develop Oct 1, 2024
1 check passed
@sohyundoh sohyundoh deleted the feat/#518 branch October 1, 2024 08:00
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
Projects
None yet
Development

Successfully merging this pull request may close these issues.

[feat] 인가 로직 도입 및 글모임 필명 GET API 수정
2 participants