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

fix(aws-serverless): Fix build of lambda layer #12083

Merged
merged 1 commit into from
May 16, 2024

Conversation

Lms24
Copy link
Member

@Lms24 Lms24 commented May 16, 2024

We previously adjusted our lambda layer auto initialization in #12017. This unfortunately changed the build output of the awslambda-auto bootstrapping script which required a package that isn't included in the layer (@sentry/node). This PR fixes the awslambda-auto file; local testing showed no more imports from @sentry/node.

fixes #12074

@@ -1,5 +1,4 @@
import { getDefaultIntegrations as getNodeDefaultIntegrations } from '@sentry/node';
import { init, tryPatchHandler } from './sdk';
import * as Sentry from './index';
Copy link
Member Author

Choose a reason for hiding this comment

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

Changed this back to how we imported the actual SDK in v7

@Lms24 Lms24 changed the title fix(aws-serverless): Fix build for lambda layer fix(aws-serverless): Fix build of lambda layer May 16, 2024
@Lms24 Lms24 merged commit 7ce2f20 into develop May 16, 2024
69 checks passed
@Lms24 Lms24 deleted the lms/fix-aws-serverless-layer-build branch May 16, 2024 15:38
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.

Incorrect v8 (241) lambda layer dependencies
2 participants