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

[20기_이한슬] Docker를 이용한 배포 미션 제출합니다. #58

Open
wants to merge 2 commits into
base: sseuldev
Choose a base branch
from

Conversation

sseuldev
Copy link

[20기_이한슬] Docker를 이용한 배포 미션 제출합니다.

Copy link

@nimikgnoej nimikgnoej left a comment

Choose a reason for hiding this comment

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

고생하셨습니다!
포스트맨 관련된 에러는 제 의견이 확실하지는 않지만 확인 한번 해보시면 될 것 같아요

README.md Outdated
Comment on lines 2160 to 2164
포스트맨을 통해 API 테스트를 진행해보았는데 `401 에러`가 발생하였다.

`signup` API의 경우, 스프링 시큐리티에서 permitAll 설정을 해주었는데도 불구하고 왜 `401 에러`가 발생할까? 응답 형태도 이상하다..

이거 왜 이러는 걸까요오..🥲

Choose a reason for hiding this comment

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

한슬님 코드를 좀 봤는데,

  1. 액세스 토큰을 넣지 않고 요청을 보냈다? -> 뭔가 아닐 것 같아요
  2. JWTFilter 에서 액세스토큰 isExpired 로 검증했는데 거기서 뭔가 에러 터지는 것 같아요.

코드를 실행해보지 않아서 정확하게는 모르겠지만 제가 보기엔 2번이 문제이지 않을까 싶습니다

Copy link
Author

Choose a reason for hiding this comment

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

찾았습니다 원인을..!!!! 제가 포스트맨으로 테스트 할 때 다른 api에서 썼던 토큰을 헤더에 그대로 포함시킨 상태로 요청을 보내고 있었더라구요.. 이걸 빼고 보냈더니 잘되었습니다.. 멍청한 이한슬이었습니다.. 같이 고민해주셔서 감사해요.. 흐엉

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.

2 participants