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

토큰 에러 검출 api 추가, 에러 메시지 내용 수정 #178

Merged
merged 11 commits into from
Nov 10, 2023

Conversation

seokwns
Copy link
Contributor

@seokwns seokwns commented Nov 10, 2023

한 일

  • GET /api/portfolio 의 모든 요청에서 토큰이 존재한다면 검증하도록 수정
  • 쿼리 스트링으로 들어오는 데이터의 타입이 일치하지 않을 경우 발생하는 에러 메시지 수정
  • 자잘한 코드 리팩토링 ...

이슈

close #176
close #177

빠이팅입니다 !

- 포트폴리오 가격 필터링 조건에 대한 변수 타입 변경
- 메서드의 변수 타입이 잘못 들어가는 경우에 대한 에러 수정
- 액세스 토큰이 유효할 경우, 토큰을 삭제하지 않고 에러만 발생하도록 변경
- 두 토큰이 모두 존재하는 요청이 들어오는 경우, 서버에 저장된 토큰쌍과 일치할 경우에만 토큰 폐기
- 변수명 형식 통일
- 이메일 인증 완료 시 30분 이내에만 사용할 수 있도록 수정
- GET /portfolio 로 들어오는 모든 요청에서 토큰이 존재한다면 검증하도록 수정
- 트랜잭션 문제로 코드 삭제
@seokwns seokwns added bug Something isn't working refactor labels Nov 10, 2023
@seokwns seokwns self-assigned this Nov 10, 2023
- 유효성 에러 메시지 수정
- 테스트 데이터 주입
Copy link
Member

@1jeongg 1jeongg left a comment

Choose a reason for hiding this comment

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

고생하셨습니다!!! 👍👍

@1jeongg 1jeongg merged commit e942dc1 into Step3-kakao-tech-campus:weekly Nov 10, 2023
1 check passed
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
bug Something isn't working refactor
Projects
None yet
Development

Successfully merging this pull request may close these issues.

쿼리스트링 데이터타입 오류 캐치 GET /api/portfolio 요청 시 토큰 검증하기
2 participants