Skip to content

🎨feat : 유저 권한 및 인증 로직 추가 - #5

Open
raymondanythings wants to merge 5 commits into
masterfrom
feature/#5-JWT-MODULE
Open

🎨feat : 유저 권한 및 인증 로직 추가#5
raymondanythings wants to merge 5 commits into
masterfrom
feature/#5-JWT-MODULE

Conversation

@raymondanythings

@raymondanythingsraymondanythings commented Feb 6, 2023

Copy link
Copy Markdown
Member

✅ PR 체크리스트

  • 코드 스타일 가이드 규칙을 준수했어요.

PR 타입

설명하실 변경 사항은 어떤 종류인가요?

  • 버그 수정
  • 기능 추가
  • 코드 스타일 업데이트
  • 기타

이슈 번호 : #5

해당 PR이 프로젝트에 큰 영향이 있나요?

  • Yes
  • No

작업 내용

  • jwt 발행 rsa 알고리즘 적용
  • socket, graphql 인증방식 적용

추가 설명

raymondanythingsand others added 5 commits February 3, 2023 19:02
@raymondanythings
🧪test : jwt module rsa 시도
30848a2
@raymondanythings
🧪test : jwt module rsa 시도
a39b141
@raymondanythings
🎨feat : DB-ORM 추가, JWT인증방식 구현
92ccb91
- 닉네임 설정로직 필요하여 로그인 로직 추가
- GQL방식 auth 로직 구현
- TODO: RSA key 저장방식 생각해봐야함
@raymondanythings
Merge branch 'master' into feature/#5-JWT-MODULE
3027466
@raymondanythings
🎨feat : 유저 권한 및 인증 로직 추가
7734732
- jwt RSA 인증 로직 추가
- socket , gql middleware 추가
@raymondanythings
raymondanythings requested a review from a team as a code ownerFebruary 6, 2023 09:02
HerrineKim
HerrineKim approved these changes Feb 6, 2023
View reviewed changes

@HerrineKimHerrineKim left a comment

Copy link
Copy Markdown

Choose a reason for hiding this comment

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

👍

Comment threadprisma/schema.prisma
Comment threadsrc/app.module.ts
Comment threadsrc/auth/auth.guard.ts
Comment threadsrc/jwt/jwt.interface.ts
cheonkyu0
cheonkyu0 reviewed Feb 6, 2023
View reviewed changes
Comment threadsrc/jwt/jwt.middleware.ts
cheonkyu0
cheonkyu0 reviewed Feb 6, 2023
View reviewed changes
Comment threadsrc/redis/redis.adapter.ts
cheonkyu0
cheonkyu0 reviewed Feb 6, 2023
View reviewed changes
Comment threadsrc/app.module.ts
ydw1996
ydw1996 approved these changes Feb 7, 2023
View reviewed changes
Comment threadsrc/auth/auth-exception.filter.ts
Comment threadsrc/jwt/jwt.service.ts
SoJeongMun
SoJeongMun approved these changes Feb 7, 2023
View reviewed changes
Comment threadsrc/app.module.ts
Sign up for freeto join this conversation on GitHub. Already have an account? Sign in to comment

Reviewers

@cheonkyu0cheonkyu0cheonkyu0 left review comments
@SoJeongMunSoJeongMunSoJeongMun approved these changes
@ydw1996ydw1996ydw1996 approved these changes
+1 more reviewer
@HerrineKimHerrineKimHerrineKim approved these changes
Reviewers whose approvals may not affect merge requirements

At least 4 approving reviews are required to merge this pull request.

Assignees

No one assigned

Labels

None yet

Projects

None yet

Milestone

No milestone

Development

Successfully merging this pull request may close these issues.

5 participants

@raymondanythings@HerrineKim@cheonkyu0@SoJeongMun@ydw1996