Skip to content

[Job Launcher] fix: do not log non-5xx errors - #3499

Merged
dnechay merged 1 commit into
developfrom
dnechay/fs-110825
Aug 11, 2025
Merged

[Job Launcher] fix: do not log non-5xx errors#3499
dnechay merged 1 commit into
developfrom
dnechay/fs-110825

Conversation

@dnechay

@dnechay dnechay commented Aug 11, 2025

Copy link
Copy Markdown
Collaborator

Issue tracking

Follow up to #3497

Context behind the change

JL in case of errors doesn't handle status code properly, which leads to wrong status code and exception logging.

How has this been tested?

  • existing tests pass (no need in extra tests, same fix as for Fortune services)

Release plan

Just merge

Potential risks; What to monitor; Rollback plan

No

@dnechay dnechay self-assigned this Aug 11, 2025
@vercel

vercel Bot commented Aug 11, 2025

Copy link
Copy Markdown

The latest updates on your projects. Learn more about Vercel for Git ↗︎

5 Skipped Deployments
Name Status Preview Comments Updated (UTC)
faucet-frontend ⬜️ Ignored (Inspect) Aug 11, 2025 10:11am
faucet-server ⬜️ Ignored (Inspect) Aug 11, 2025 10:11am
human-app ⬜️ Skipped (Inspect) Aug 11, 2025 10:11am
human-dashboard-frontend ⬜️ Skipped (Inspect) Aug 11, 2025 10:11am
staking-dashboard ⬜️ Skipped (Inspect) Aug 11, 2025 10:11am

@dnechay
dnechay requested a review from Dzeranov August 11, 2025 10:11
@dnechay dnechay mentioned this pull request Aug 11, 2025
15 tasks
@dnechay
dnechay merged commit 067c473 into develop Aug 11, 2025
13 checks passed
@dnechay
dnechay deleted the dnechay/fs-110825 branch August 11, 2025 10:14
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