Commit e1b617a

committed

review

1 parent e72b0ea commit e1b617aCopy full SHA for e1b617a

File tree

+14

-9

lines changed

packages/aws-serverless/src
- init.ts
- lambda-extension
  - aws-lambda-extension.ts

+14

-9

lines changed

Lines changed: 14 additions & 3 deletions

Original file line number	Diff line number	Diff line change
`@@ -36,9 +36,20 @@ export function init(options: AwsServerlessOptions = {}): NodeClient \| undefined`
`36`	`36`
`37`	`37`	`const sdkSource = getSDKSource();`
`38`	`38`
`39`		`- if (opts._experiments?.enableLambdaExtension && sdkSource === 'aws-lambda-layer' && !opts.tunnel) {`
`40`		`- DEBUG_BUILD && debug.log('Proxying Sentry events through the Sentry Lambda extension');`
`41`		`- opts.tunnel = 'http://localhost:9000/envelope';`
	`39`	`+ if (opts._experiments?.enableLambdaExtension) {`
	`40`	`+ if (sdkSource === 'aws-lambda-layer') {`
	`41`	`+ if (!opts.tunnel) {`
	`42`	`+ DEBUG_BUILD && debug.log('Proxying Sentry events through the Sentry Lambda extension');`
	`43`	`+ opts.tunnel = 'http://localhost:9000/envelope';`
	`44`	`+ } else {`
	`45`	`+ DEBUG_BUILD &&`
	`46`	`+ debug.warn(`
	`47`	+ `Using a custom tunnel with the Sentry Lambda extension is not supported. Events will be tunnelled to ${opts.tunnel} and not through the extension.`,
	`48`	`+ );`
	`49`	`+ }`
	`50`	`+ } else {`
	`51`	`+ DEBUG_BUILD && debug.warn('The Sentry Lambda extension is only supported when using the AWS Lambda layer.');`
	`52`	`+ }`
`42`	`53`	`}`
`43`	`54`
`44`	`55`	`applySdkMetadata(opts, 'aws-serverless', ['aws-serverless'], sdkSource);`

Lines changed: 0 additions & 6 deletions

Original file line number	Diff line number	Diff line change
`@@ -55,12 +55,6 @@ export class AwsLambdaExtension {`
`55`	`55`	`if (!res.ok) {`
`56`	`56`	throw new Error(`Failed to advance to next event: ${await res.text()}`);
`57`	`57`	`}`
`58`		`-`
`59`		`- const event = (await res.json()) as { eventType: string };`
`60`		`-`
`61`		`- if (event.eventType === 'SHUTDOWN') {`
`62`		`- await new Promise(resolve => setTimeout(resolve, 1000));`
`63`		`- }`
`64`	`58`	`}`
`65`	`59`
`66`	`60`	`/**`

Comments

(0)