Skip to content
New issue

Have a question about this project? Sign up for a free GitHub account to open an issue and contact its maintainers and the community.

By clicking “Sign up for GitHub”, you agree to our terms of service and privacy statement. We’ll occasionally send you account related emails.

Already on GitHub? Sign in to your account

1040 add defaultHandler to lambdas #1357

Draft
wants to merge 1 commit into
base: develop
Choose a base branch
from

Conversation

leite08
Copy link
Member

@leite08 leite08 commented Dec 19, 2023

Ref: metriport/metriport-internal#1040

Dependencies

none

Description

Add defaultHandler() to lambdas so we can let error/exceptions "bubble up" all the way to the default handler and it will make sure the additional info from MetriportError-like errors will be included on the Sentry event.

Testing

  • Local
    • Branch to staging, try to download an invalid file, see if the error gets the additional context (cwReferenceHeader and documentLocation)
  • Staging
    • none
  • Production
    • none

Release Plan

  • nothing special

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.

None yet

1 participant