fix: 쿠키 적용 후 개발서버에서 에러 발생하는 문제 해결 #259

uwoobeat · 2024-01-30T16:14:18Z

🌱 관련 이슈

📌 작업 내용 및 특이사항

스웨거 시큐리티 스키마 활성화 처리해서 API 요청 날릴 떄 헤더 포함되도록 세팅했습니다.
헤더 가져와서 Bearer prefix 떼주는 부분에서 NPE 발생하는 이슈 있어서, Optional로 감싸줬습니다.
- p.s. 오늘 서버에서 503 터지는 이슈 있어서 서버 로그 확인해보니 누군가 NPE를 마구마구 터트리고 있어서... 서버가 공격당하는 줄 알고 식겁했지만... 알고보니 헬스체크를 정기적으로 날리는 LB였고... 헬스체크 API에서 NPE가 터지고 있어서 해당 이슈가 발생하는 거였습니다...
시큐리티 유틸리티에서 익명 유저로 로그인하는 경우 getName() 호출할 때 memberId가 아니라 문자열이 들어오기 때문에, try-catch로 감싸줬습니다.
또한 시큐리티 유틸에서 PrincipalDetails 로 캐스팅하는 방법 대신 Authentication 인터페이스가 제공하는 getName() 사용해서 좀 더 우아하게 처리해줬습니다.

📝 참고사항

📚 기타

sonarqubecloud · 2024-01-31T12:50:17Z

Quality Gate passed

The SonarCloud Quality Gate passed, but some issues were introduced.

69 New issues
0 Security Hotspots
51.4% Coverage on New Code
0.0% Duplication on New Code

See analysis details on SonarCloud

char-yb

LGTM
고생많았습니당:)

uwoobeat added 2 commits January 31, 2024 01:13

fix: 캐스팅 대신 인터페이스 메서드 사용하도록 수정

835c062

chore: 테스트 트리거 활성화

e06027c

uwoobeat temporarily deployed to DEV January 30, 2024 16:14 — with GitHub Actions Inactive

uwoobeat temporarily deployed to DEV January 30, 2024 16:16 — with GitHub Actions Inactive

fix: 엑세스 토큰을 헤더에서 추출하는 로직 수정

972084f

uwoobeat temporarily deployed to DEV January 30, 2024 16:30 — with GitHub Actions Inactive

uwoobeat temporarily deployed to DEV January 30, 2024 16:32 — with GitHub Actions Inactive

refactor: 스웨거 리프레시 토큰 스키마 제거

c8d1a38

uwoobeat temporarily deployed to DEV January 31, 2024 03:46 — with GitHub Actions Inactive

uwoobeat temporarily deployed to DEV January 31, 2024 03:48 — with GitHub Actions Inactive

fix: 헤더가 null인 경우 파싱하기 전에 Optional로 감싸도록 수정

45183a0

uwoobeat temporarily deployed to DEV January 31, 2024 03:59 — with GitHub Actions Inactive

uwoobeat temporarily deployed to DEV January 31, 2024 04:01 — with GitHub Actions Inactive

fix: 헤더 변환 로직 수정

37b8170

uwoobeat temporarily deployed to DEV January 31, 2024 04:10 — with GitHub Actions Inactive

uwoobeat temporarily deployed to DEV January 31, 2024 04:12 — with GitHub Actions Inactive

fix: 시큐리티 유틸이 멤버 ID 파싱에 실패할 경우 커스텀 예외 던지도록 수정

6d5f833

uwoobeat temporarily deployed to DEV January 31, 2024 04:42 — with GitHub Actions Inactive

uwoobeat temporarily deployed to DEV January 31, 2024 04:44 — with GitHub Actions Inactive

feat: 스웨거 API 요청 시 엑세스 토큰 헤더에 포함되도록 설정 변경

197be9d

uwoobeat temporarily deployed to DEV January 31, 2024 05:01 — with GitHub Actions Inactive

uwoobeat temporarily deployed to DEV January 31, 2024 05:03 — with GitHub Actions Inactive

chore: 테스트 트리거 비활성화

02fe1b3

uwoobeat marked this pull request as ready for review January 31, 2024 12:48

uwoobeat requested review from kdomo and char-yb as code owners January 31, 2024 12:48

char-yb approved these changes Jan 31, 2024

View reviewed changes

uwoobeat merged commit c4c7703 into develop Jan 31, 2024
2 checks passed

github-actions bot added the merged 머지된 PR label Jan 31, 2024

Provide feedback

Saved searches

Use saved searches to filter your results more quickly

fix: 쿠키 적용 후 개발서버에서 에러 발생하는 문제 해결 #259

fix: 쿠키 적용 후 개발서버에서 에러 발생하는 문제 해결 #259

uwoobeat commented Jan 30, 2024 •

edited

Loading

sonarqubecloud bot commented Jan 31, 2024

char-yb left a comment

fix: 쿠키 적용 후 개발서버에서 에러 발생하는 문제 해결 #259

fix: 쿠키 적용 후 개발서버에서 에러 발생하는 문제 해결 #259

Conversation

uwoobeat commented Jan 30, 2024 • edited Loading

🌱 관련 이슈

📌 작업 내용 및 특이사항

📝 참고사항

📚 기타

sonarqubecloud bot commented Jan 31, 2024

Quality Gate passed

char-yb left a comment

Choose a reason for hiding this comment

uwoobeat commented Jan 30, 2024 •

edited

Loading