Skip to content
This repository was archived by the owner on Jan 29, 2026. It is now read-only.

middleware 내부에서 httpError 로깅 추가 및 x-soto-session 제거 로직 삭제#3649

Merged
guswl98 merged 2 commits intoepic/KLZT-856_nol_userfrom
fix/middleware
Apr 8, 2025
Merged

middleware 내부에서 httpError 로깅 추가 및 x-soto-session 제거 로직 삭제#3649
guswl98 merged 2 commits intoepic/KLZT-856_nol_userfrom
fix/middleware

Conversation

@guswl98
Copy link
Copy Markdown
Contributor

@guswl98 guswl98 commented Apr 8, 2025

PR 설명

당분간 x-soto-session을 유지하기로 했으므로 x-soto-session을 제외하는 로직을 삭제합니다.
미들웨어의 에러 트래킹을 위해 captureHttpError를 추가합니다.

@guswl98 guswl98 self-assigned this Apr 8, 2025
@codecov
Copy link
Copy Markdown

codecov bot commented Apr 8, 2025

Codecov Report

Attention: Patch coverage is 16.66667% with 5 lines in your changes missing coverage. Please review.

Project coverage is 10.05%. Comparing base (c957d85) to head (4ae1604).
Report is 2 commits behind head on epic/KLZT-856_nol_user.

Files with missing lines Patch % Lines
.../react-contexts/src/middlewares/refresh-session.ts 16.66% 5 Missing ⚠️
Additional details and impacted files
@@                    Coverage Diff                     @@
##           epic/KLZT-856_nol_user    #3649      +/-   ##
==========================================================
- Coverage                   10.05%   10.05%   -0.01%     
==========================================================
  Files                         741      741              
  Lines                       29208    29204       -4     
  Branches                     7932     7930       -2     
==========================================================
- Hits                         2937     2936       -1     
+ Misses                      26251    26248       -3     
  Partials                       20       20              

☔ View full report in Codecov by Sentry.
📢 Have feedback on the report? Share it here.

🚀 New features to boost your workflow:
  • ❄️ Test Analytics: Detect flaky tests, report on failures, and find test suite problems.
  • 📦 JS Bundle Analysis: Save yourself from yourself by tracking and limiting bundle sizes in JS merges.

@guswl98
Copy link
Copy Markdown
Contributor Author

guswl98 commented Apr 8, 2025

release-canary

@github-actions
Copy link
Copy Markdown

github-actions bot commented Apr 8, 2025

v13.45.2-pr-3649.79 has been published!

@guswl98 guswl98 requested a review from dongoc April 8, 2025 08:53
@guswl98 guswl98 marked this pull request as ready for review April 8, 2025 08:54
@guswl98 guswl98 requested a review from a team as a code owner April 8, 2025 08:54
@guswl98 guswl98 requested review from choisohyun and dia-triple and removed request for a team April 8, 2025 08:54
@guswl98 guswl98 merged commit 2dd1caa into epic/KLZT-856_nol_user Apr 8, 2025
13 of 16 checks passed
@guswl98 guswl98 deleted the fix/middleware branch April 8, 2025 09:02
Sign up for free to subscribe to this conversation on GitHub. Already have an account? Sign in.

Projects

None yet

Development

Successfully merging this pull request may close these issues.

2 participants