Skip to content

feat: improve unhandled exceptions logging - #3511

Merged
dnechay merged 2 commits into
developfrom
dnechay/fs-200825
Aug 21, 2025
Merged

feat: improve unhandled exceptions logging#3511
dnechay merged 2 commits into
developfrom
dnechay/fs-200825

Conversation

@dnechay

@dnechay dnechay commented Aug 20, 2025

Copy link
Copy Markdown
Collaborator

Issue tracking

Freestyle.

Context behind the change

  • added request url to unhandled exception logs so we can see on which route it takes place
  • now logging of unhandled Axios errors in HUMAN App also includes the outgoing request url, so we can see what is the external resource that might have potential issue (recently we got alerts about CA errors, but can't see what is the source)

How has this been tested?

  • existing unit tests build and pass; no need for extra tests since it just changes logs format

Release plan

Just merge

Potential risks; What to monitor; Rollback plan

No

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

vercel Bot commented Aug 20, 2025

Copy link
Copy Markdown

The latest updates on your projects. Learn more about Vercel for GitHub.

5 Skipped Deployments
Project Deployment Preview Comments Updated (UTC)
faucet-frontend Ignored Ignored Aug 20, 2025 8:40am
faucet-server Ignored Ignored Aug 20, 2025 8:40am
human-app Skipped Skipped Aug 20, 2025 8:40am
human-dashboard-frontend Skipped Skipped Aug 20, 2025 8:40am
staking-dashboard Skipped Skipped Aug 20, 2025 8:40am

@dnechay dnechay moved this to In Review in Core-tech - 2025 Aug 20, 2025
@dnechay
dnechay requested a review from flopez7 August 20, 2025 08:40
@dnechay dnechay mentioned this pull request Aug 20, 2025
10 tasks
@dnechay
dnechay merged commit cf71981 into develop Aug 21, 2025
17 checks passed
@dnechay
dnechay deleted the dnechay/fs-200825 branch August 21, 2025 08:01
@github-project-automation github-project-automation Bot moved this from In Review to Done in Core-tech - 2025 Aug 21, 2025
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Labels

None yet

Projects

Status: Done

Development

Successfully merging this pull request may close these issues.

2 participants