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.

Sign up for GitHub

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

Jump to bottom

논의한 부분들에 대한 리팩토링 및 버그 수정 #64

Closed
2 tasks done
devphilip21 opened this issue Nov 28, 2019 · 0 comments
Closed
2 tasks done

논의한 부분들에 대한 리팩토링 및 버그 수정 #64

devphilip21 opened this issue Nov 28, 2019 · 0 comments
Assignees
@devphilip21
Labels
🐣Feature 새로운 기능을 추가할 때 사용하는 라벨 📠Location : converter server converter 서버에 대한 작업을 나타내는 라벨 📠Location : web server web 서버에 대한 작업을 나타내는 라벨
Projects
Sprint - 03 🏃‍♂️
Milestone
Sprint - 03

Comments

@devphilip21
Copy link
Contributor

devphilip21 commented Nov 28, 2019 •
edited
Loading

논의한 부분들에 대한 리팩토링 및 버그 수정

  • auth 관련 로직 hook 사용방식 재정리해서 리팩토링
  • 이미지 컨버터 서버에서 익명토큰에 대한 verify 확인

완료 조건 ✅

  • 이미지 컨버터는 로그인한 사용자만 이용 가능하다.
  • auth 관련 로직 hook 사용방식 재정리해서 리팩토링

관련 이슈 📎

레퍼런스 📚

레퍼런스 없음
@devphilip21 devphilip21 added 🐣Feature 새로운 기능을 추가할 때 사용하는 라벨 📠Location : web server web 서버에 대한 작업을 나타내는 라벨 📠Location : converter server converter 서버에 대한 작업을 나타내는 라벨 labels Nov 28, 2019
@devphilip21 devphilip21 added this to the Sprint - 03 milestone Nov 28, 2019
@devphilip21 devphilip21 self-assigned this Nov 28, 2019
devphilip21 pushed a commit that referenced this issue Nov 28, 2019
[ FEATURE ] 익명의 사용자의 경우 이미지 컨버터 서버에서 권한 오류 응답 (#64)
53a7977 
Why
- 익명의 사용자는 채널을 생성할 수 있는 권한이 없다.

What
- token 에 isAnonymous Flag 정보를 적재한다.
- 이미지 컨버터 서버에서 익명의 사용자인지 확인한다.
devphilip21 pushed a commit that referenced this issue Nov 28, 2019
[ REFACTOR ] 클라이언트 인증 관련 로직 리팩토링 및 버그 수정 (#64)
84489b4 
Why
- 인증관련 로직을 apollo 가 이해도가 낮을 때 만들었는데, 현재 더 나은방식으로 리팩토링 가능하다.
- 이미지 컨버터 서버에 현재 수정된 인증관련 로직이 반영되지 않았다.

What
- 이미지 컨버터 서버 인증 로직 수정
- apollo client 캐싱 사용방식 수정
- apollo client hook 클러스터링
@devphilip21 devphilip21 mentioned this issue Nov 28, 2019
Merged
@always-awake always-awake mentioned this issue Nov 28, 2019
Merged
@devphilip21 devphilip21 moved this from To do to Done in Sprint - 03 🏃‍♂️ Nov 30, 2019
@aereeeee aereeeee mentioned this issue Dec 2, 2019
Closed
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Assignees

@devphilip21 devphilip21

Labels
🐣Feature 새로운 기능을 추가할 때 사용하는 라벨 📠Location : converter server converter 서버에 대한 작업을 나타내는 라벨 📠Location : web server web 서버에 대한 작업을 나타내는 라벨
Projects
No open projects
Sprint - 03 🏃‍♂️
Done
Milestone
Sprint - 03
Development

No branches or pull requests
1 participant
@devphilip21