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

feature: Swagger API 명세 추가 #50

Merged
merged 14 commits into from
Feb 6, 2024

Conversation

Combi153
Copy link
Contributor

@Combi153 Combi153 commented Feb 6, 2024

📌 관련 이슈

📁 작업 설명

  • Swagger API 명세 설정 추가했습니다!

참고

Swagger에서 Auth 관련 API 들은 테스트할 수 없거나 힘듭니다. 따라서 Auth 관련 API 들은 Hidden 처리했습니다.

1. /auth/Kakao

redirect 페이지를 로드할 수 없습니다. CORS 에러를 마주하게 됩니다.
image

2. /auth/login/Kakao

카카오에서 전달해주는 auth code를 받을 수 없습니다.

3. /auth/token

Swagger UI에서 refreshToken 을 서버에게 cookie로 전달할 수 없습니다. 다음 자료를 참고해주세요.

Note for Swagger UI and Swagger Editor users: Cookie authentication is currently not supported for "try it out" requests due to browser security restrictions. See this issue for more information. SwaggerHub does not have this limitation.

기타

  • 각자 localhost 에서 테스트 해보셔도 좋을 것 같습니다!

Copy link
Member

@TaeyeonRoyce TaeyeonRoyce left a comment

Choose a reason for hiding this comment

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

스웨거도 이렇게 구성하니깐 UI가 깔끔하고 잘 보여지네요!
너무 고생하셨습니다..!👍

Copy link

github-actions bot commented Feb 6, 2024

Test Results

 20 files  ±0   20 suites  ±0   6s ⏱️ ±0s
 71 tests ±0   71 ✅ ±0  0 💤 ±0  0 ❌ ±0 
102 runs  ±0  102 ✅ ±0  0 💤 ±0  0 ❌ ±0 

Results for commit fe15821. ± Comparison against base commit 7971059.

♻️ This comment has been updated with latest results.

Copy link
Contributor

@hgo641 hgo641 left a comment

Choose a reason for hiding this comment

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

스웨거 처음 써봐서 신기하네요! 어프루브합니다!!

@Combi153 Combi153 force-pushed the 30-feature-swagger-api-명세-추가 branch from 0802233 to fe15821 Compare February 6, 2024 06:28
@Combi153 Combi153 merged commit 6745cfc into develop Feb 6, 2024
2 checks passed
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.

feature: Swagger API 명세 추가
3 participants