Skip to content

fix(hono): Include originalException in captured exception hint - #22979

Merged
Lms24 merged 1 commit into
getsentry:developfrom
kamilogorek:hono-originalexception
Aug 4, 2026
Merged

fix(hono): Include originalException in captured exception hint#22979
Lms24 merged 1 commit into
getsentry:developfrom
kamilogorek:hono-originalexception

Conversation

@kamilogorek

@kamilogorek kamilogorek commented Aug 4, 2026

Copy link
Copy Markdown
Contributor

Hono is the only package that is using client interface directly. Because of that, none of default behavior applies, and captureException is not receiving a standard originalException property.
Without it, LinkedErrors integration and alike do not work and any data about cause chain is lost.

@kamilogorek
kamilogorek requested a review from a team as a code owner August 4, 2026 08:53
@kamilogorek
kamilogorek requested review from nicohrubec and s1gr1d and removed request for a team August 4, 2026 08:53
Lms24
Lms24 approved these changes Aug 4, 2026
View reviewed changes

@Lms24 Lms24 left a comment

Copy link
Copy Markdown
Member

Choose a reason for hiding this comment

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

Hey Kamil 👋
Change LGTM and is a minimal fix for the time being.

cc @s1gr1d once you're back, is there a reason for using the client rather than the scope's captureException?

@Lms24
Lms24 merged commit 43d55a9 into getsentry:develop Aug 4, 2026
84 of 86 checks passed
@javascript-sdk-gitflow javascript-sdk-gitflow Bot mentioned this pull request Aug 4, 2026
Closed
@Lms24 Lms24 mentioned this pull request Aug 4, 2026
Merged
@kamilogorek
kamilogorek deleted the hono-originalexception branch August 4, 2026 09:47
Lms24 added a commit that referenced this pull request Aug 4, 2026
@Lms24 @kamilogorek
fix(v10/hono): Include originalException in captured exception hint (#…
84c9dfe 
…22990)

Backport of: #22979

Co-authored-by: Kamil Ogórek <kamil.ogorek@gmail.com>
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Reviewers

@Lms24 Lms24 Lms24 approved these changes
@JPeer264 JPeer264 JPeer264 approved these changes
@msonnb msonnb msonnb approved these changes
@nicohrubec nicohrubec nicohrubec approved these changes
@andreiborza andreiborza andreiborza approved these changes
@s1gr1d s1gr1d Awaiting requested review from s1gr1d s1gr1d is a code owner automatically assigned from getsentry/team-javascript-sdks-framework

Assignees

No one assigned

Labels

None yet

Projects

None yet

Milestone

No milestone

Development

Successfully merging this pull request may close these issues.

6 participants

@kamilogorek @Lms24 @JPeer264 @msonnb @nicohrubec @andreiborza