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

Feat 스페이스 생성 api #14

Merged
merged 4 commits into from
Jul 17, 2024
Merged

Feat 스페이스 생성 api #14

merged 4 commits into from
Jul 17, 2024

Conversation

seongjunnoh
Copy link
Collaborator

📝 요약

이슈 번호 : #12

🔖 변경 사항

스페이스 생성 api 개발
로그인 진행한 유저는 jwt를 발급받아서 스페이스 생성 요청을 보낼 수 있음 -> 로그인 response의 헤더에 있는 jwt를 스페이스 생성 request 헤더에 넣어야 함
스페이스 생성 요청에 대한 response의 헤더에는 새로운 jwt가 발급됨 (이 유저가 해당 스페이스에 속해있음을 알려주는 jwt)
-> 새로 발급되는 jwt에는 userId, <spaceId, 해당 스페이스에서 이 유저의 권한> map의 list 가 payload에 들어있음
-> 이 정보를 가지고 해당 스페이스에 대한 요청을 보낼수 있는 권한을 가지고있는지를 확인할수 있습니다!

✅ 리뷰 요구사항

📸 확인 방법 (선택)

  1. postMan에서 회원가임 & 로그인 요청을 보내고, 로그인 응답의 헤더에 들어있는 jwt(loginJwt)를 확보
  2. loginJwt를 스페이스 생성 요청의 헤더에 주입
  3. 스페이스 생성 응답의 헤더에 들어있는 jwt(UserSpaceJwt)를 확보
  4. userSpaceJwt를 test용 요청의 헤더에 주입(test용 요청은 해당 스페이스에 대한 권한을 가진 유저만이 요청을 보낼 수 있도록 확인하기 위한 용도임)
  5. 200 ok 뜨는지 확인


📌 PR 진행 시 이러한 점들을 참고해 주세요

* P1 : 꼭 반영해 주세요 (Request Changes) - 이슈가 발생하거나 취약점이 발견되는 케이스 등
* P2 : 반영을 적극적으로 고려해 주시면 좋을 것 같아요 (Comment)
* P3 : 이런 방법도 있을 것 같아요~ 등의 사소한 의견입니다 (Chore)

@seongjunnoh seongjunnoh linked an issue Jul 17, 2024 that may be closed by this pull request
2 tasks
@drbug2000
Copy link
Collaborator

LGTM

1 similar comment
@arkchive
Copy link
Collaborator

LGTM

Copy link
Member

@hyunn522 hyunn522 left a comment

Choose a reason for hiding this comment

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

LGTM!

@seongjunnoh seongjunnoh merged commit 47fc76e into develop Jul 17, 2024
3 checks passed
@hyunn522 hyunn522 deleted the 12-스페이스-생성 branch July 25, 2024 12:29
seongjunnoh added a commit that referenced this pull request Oct 30, 2024
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.

스페이스 생성
4 participants