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/add blockchain #25

Merged
merged 10 commits into from
Aug 26, 2024
Merged

feat/add blockchain #25

merged 10 commits into from
Aug 26, 2024

Conversation

cozy-hn
Copy link
Contributor

@cozy-hn cozy-hn commented Aug 26, 2024

주요 구현 사항

  • 블록체인을 이용한 tournament 기록 읽기 기능 했습니다.
  • logger 추가했습니다.
  • exception custom 해서 넣어놨습니다.

리뷰어 참고 사항

  • logger는 setting에서 단계를 조절하면 해당 단계 아래의 로드는 나오지 않습니다. printf문을 안지워도 됩니다.
  • 단계는 DEBUG, INFO, 'WARNING', 'ERROR', 'CRITICAL'있습니다.
  • 로거는 장고자체에서도 사용되어 있서 가끔 더 쓸만한정보가 튀어나온답니다.
  • 사용법은 클래스인 경우 self.logger = logging.getLogger(self.__class__.__name__) 함수인 경우 logger = logging.getLogger(my_function.__name__)으로 사용하면 된답니다.
  • exception은 Response로 관리하면 너무 길어서 따로 관리해주는게 좋을것 같아요. 제가 어느정도 커스텀해놔서 이쁘게 나온답니다.
  • 그리고 에러도 response return도 해줘야 프론트에서 디버깅이 쉽습니다.
  • 에러 return이 아니라 바로 raise해도 되용

추가적인 말

아무것도 안한 주제에... 너가 무슨 말이냐

  • util 폴더를 따로 만들었는데 공용으로 쓰는 함수 이쪽으로 박아주세요. get user id나 이런거?
  • jwt를 인증절차를 앞으로 거의 모든 api에서 매번 부를거고 현재도 그러고 있는데 클래스 세팅으로 인증 확인 할 수 있는걸로 알고 있어요
  • jwt를 발급하는 api를 스웨거에서 만들수 있게 임시로 만들면 여러 유저를 확인하기 쉬울 것 같아요(만들시 존재하는 jwt는 지워야함)
  • 쿠키도 스웨거에서 같이 보낼 수 있는걸로 알고있는데 바로 인증도 시킬 수 있는지 확인해봐야 할 것 같아요.
  • 스웨거 description적어주세요 무슨 api인지 하나도 모르겠어요 ㅠㅠ
  • test코드 작성의 필요성이 시급해보입니다. (나도 안적었지만) 이미 늦었지만 쌓아두고 적으면 죽을 맛일거에요.
  • 딕셔너리, json에 get 쓸 때 두번째 인자 None으로 둬주세요 안두면 그냥 에러난답니다!

@seonjo1
Copy link
Contributor

seonjo1 commented Aug 26, 2024

고생했다!! 편히 쉬거라..!!

@jaehejun
Copy link
Contributor

jwt를 인증절차를 앞으로 거의 모든 api에서 매번 부를거고 현재도 그러고 있는데 클래스 세팅으로 인증 확인 할 수 있는걸로 알고 있어요

  • 이게 JWT 미들웨어를 말하시는 건가요!?

test코드 작성의 필요성이 시급해보입니다. (나도 안적었지만) 이미 늦었지만 쌓아두고 적으면 죽을 맛일거에요.

  • test코드는 어떤식으로 작성해야하는지 도움 주시면 감사하겠습니다...

고생하셨습니다 클러가서 찬찬히 읽어보고 pr 리뷰 하겠읍니다!

@seonjo1
Copy link
Contributor

seonjo1 commented Aug 26, 2024

오늘 고지흔 영업 마감입니다!! 질문에 대한 답변은 다음 기회에!!

@cozy-hn
Copy link
Contributor Author

cozy-hn commented Aug 26, 2024

@jaehejun
추가영업합니다.
jwt 검증 방식을 조사해봤는데 1. 미들웨어 2. 헤더 Authorization으로 빼고 simplejwt쓴다 3. util이나 class마다 검증함수 넣고 돌려쓴다 정도가 되겠네요
test코드는 기능이나 함수 단위에 있는 정상작동 + 에러 다 만들어서 그 에러가 맞는지 확인하면되고 인자가 없거나 타입이 안맞는 경우 나 넣으면서 확인 하면되는데 이걸 gpt와 코파일럿이 참 잘한답니다.

@cozy-hn
Copy link
Contributor Author

cozy-hn commented Aug 26, 2024

@jaehejun + 목업이라고 있는데... 이것도 쓰면 좋아요

@seonjo1 seonjo1 merged commit 3a0e0e7 into main Aug 26, 2024
2 checks passed
@seonjo1 seonjo1 deleted the feat/add_blockchain branch August 26, 2024 08:39
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.

3 participants