Skip to content

Conversation

msonnb
Copy link
Contributor

@msonnb msonnb commented Sep 11, 2025

Sentry.AWSLambda.init

Looks like this hasn't been updated in a while 👀

@msonnb msonnb requested a review from andreiborza September 11, 2025 15:43
@msonnb msonnb self-assigned this Sep 11, 2025
throw new Error('oh, hello there!');
});
};
Copy link

Choose a reason for hiding this comment

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

Bug: Sentry Integration Missing in Examples

The manual Sentry initialization examples in the README.md removed Sentry.wrapHandler(). This wrapper is crucial for Sentry to automatically capture exceptions, manage the Lambda execution context, and provide performance tracing. Without it, these examples won't properly integrate Sentry.

Fix in Cursor Fix in Web

Copy link
Member

@Lms24 Lms24 left a comment

Choose a reason for hiding this comment

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

Nice! Yeah, looks like this still comes from @sentry/serverless days. Thanks for updating!

@msonnb msonnb merged commit 636b2a5 into develop Sep 11, 2025
38 checks passed
@msonnb msonnb deleted the ms/aws-readme branch September 11, 2025 16:07
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