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

HOTFIX: health check 용 api 구현 #225

Merged
merged 2 commits into from
Sep 21, 2024
Merged

HOTFIX: health check 용 api 구현 #225

merged 2 commits into from
Sep 21, 2024

Conversation

adorableco
Copy link
Contributor

@adorableco adorableco commented Sep 21, 2024

PR

✨ 작업 내용

스크린샷 2024-09-21 오후 7 00 03
  • 지속적으로 발생하는 access denied의 원인이 apm 이 주기적으로 "/" 엔드포인트로 헬스 체크를 하기 때문이었습니다..! 기존에는 auth 관련 엔드포인트를 제외한 모든 엔드포인트가 authenticated 확인이 필요했으므로 access denied 가 발생하는 것이었습니다.
    • 따라서 해당 부분을 anonymous 로 설정했습니다.
  • 더하여 "/" 엔드 포인트를 헬스 체크 엔드 포인트로 지정하여 response를 반환하도록 해서 no static resource 오류가 발생하는 것을 방지했습니다.
  • 리뷰 없이 머지했던 부분에서 헤더 값으로 ELB-HealthChecker/2.0 이 있는지 체크하여, 존재하는 경우 token validation 을 패스하도록 했는데 위와 같이 anonymous 로 지정해주기만 하면돼서 해당 부분은 삭제했습니다.
  • 테스트는 로컬에서 직접 도커 빌드해서 푸시하는 방법으로 완료했습니다..!

✨ 참고 사항

  • 이제 error rate 100% 아니당..

⏰ 현재 버그

x


✏ Git Close

@adorableco adorableco self-assigned this Sep 21, 2024
@adorableco adorableco added 🛠️ Fix Something isn't working 🫧 seyeon seyeon labels Sep 21, 2024
Copy link

테스트 커버리지 측정

There is no coverage information present for the Files changed

Total Project Coverage 83.99% 🍏

@swm-standard swm-standard deleted a comment from github-actions bot Sep 21, 2024
Copy link
Contributor

@RinRinPARK RinRinPARK left a comment

Choose a reason for hiding this comment

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

우리 서버 에러는 0%야 멍총한 뉴렐릭녀석

@adorableco adorableco merged commit 8157aa6 into develop Sep 21, 2024
1 check passed
@adorableco adorableco deleted the seyeon/swm-180 branch October 9, 2024 01:20
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
🛠️ Fix Something isn't working 🫧 seyeon seyeon
Projects
None yet
Development

Successfully merging this pull request may close these issues.

2 participants