woowacourse-teams / 2023-festa-go

🎪 페스타고, 대학 축제를 더욱 즐겁게!
71 stars 8 forks source link

[BE] fix: Swagger 문서에 표시되는 memberId 파라미터 삭제 및 인증 필요한 API 표식 추가, 레거시 코드 삭제 (#918) #919

Closed seokjin8678 closed 4 months ago

seokjin8678 commented 4 months ago

📌 관련 이슈

✨ PR 세부 내용

@Member 어노테이션에 @Parameter(hidden = true)을 추가하여 파리미터에 표시되는 memberId를 삭제했습니다.

또한 인증이 필요한 API의 경우에도 @MemberAuth 어노테이션에 @SecurityRequirement(name = "bearerAuth")를 추가하여 인증이 필요한 API를 표시하였습니다.

다행이 사용되는 어노테이션에 붙여도 스웨거가 잘 인식해서 노가다할 걱정이 줄었네요. 😂

매 컨트롤러 메서드마다 어노테이션 붙일 생각하니 아찔했습니다.

image

추가로 새로운 V1 인증 기능이 잘 정착해서 기존 레거시 인증 Controller, Service 클래스는 삭제했습니다.

github-actions[bot] commented 4 months ago

Test Results

214 files  214 suites   26s :stopwatch: 704 tests 704 :white_check_mark: 0 :zzz: 0 :x: 717 runs  717 :white_check_mark: 0 :zzz: 0 :x:

Results for commit d80836dc.

seokjin8678 commented 4 months ago

기능 추가가 아닌, 수정에 대한 기능이므로 리뷰 기다리지 않고 머지하겠습니다!