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

[fix] #212 - 커스텀 어노테이션 CurrentMember 파라미터 옵션 hidden으로 설정 #213

Merged
merged 1 commit into from
Sep 26, 2024

Conversation

hoonyworld
Copy link
Member

Related issue 🛠

Work Description ✏️

  • @CurrentMember는 컨트롤러 메서드의 파라미터로 선언되지만, 실제로는 클라이언트가 값을 직접 제공하는 것이 아니라, 서버에서 자동으로 해당 값을 추출하여 넣어주는 역할을 합니다.
  • 따라서 클라이언트가 API 요청을 보낼 때, @CurrentMember로 지정된 파라미터에 값을 직접 전달하지 않도록 해야합니다.
  • 하지만 Swagger에서는 직접 파라미터를 전달하도록 구현이 되어있었고, 클라이언트 측에서 테스트를 할 수 없었습니다.
  • 이를 해결하기 위해, Swagger에서 해당 파라미터를 hidden 옵션을 통해 명시하지 않도록 구현했습니다.

Trouble Shooting ⚽️

Related ScreenShot 📷

image
  • memberId를 파라미터로 주는 부분이 Swagger에 명시가 되지 않는 것을 확인했습니다.
  • 이 밖에 다른 API에서도 모두 확인했습니다!

Uncompleted Tasks 😅

To Reviewers 📢

Copy link
Collaborator

@hyerinhwang-sailin hyerinhwang-sailin left a comment

Choose a reason for hiding this comment

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

스웨거 정상화 좋습니다~

@hoonyworld hoonyworld merged commit 7074729 into develop Sep 26, 2024
1 check passed
@hoonyworld hoonyworld deleted the bug/#212 branch September 26, 2024 14:58
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.

[bug] @CurrentMember 커스텀 어노테이션, 스웨거에서 parameter가 보이는 문제
2 participants