remove rethrowAfterCapture option

Author: lobsterkatie

packages/serverless/src/awslambda.ts

@@ -42,7 +42,6 @@ export type AsyncHandler<T extends Handler> = (
 
 export interface WrapperOptions {
   flushTimeout: number;
-  rethrowAfterCapture: boolean;
   callbackWaitsForEmptyEventLoop: boolean;
   captureTimeoutWarning: boolean;
   timeoutWarningLimit: number;
@@ -215,11 +214,10 @@ function enhanceScopeWithEnvironmentData(scope: Scope, context: Context, startTi
 export function wrapHandler<TEvent, TResult>(
   handler: Handler<TEvent, TResult>,
   wrapOptions: Partial<WrapperOptions> = {},
-): Handler<TEvent, TResult | undefined> {
+): Handler<TEvent, TResult> {
   const START_TIME = performance.now();
   const options: WrapperOptions = {
     flushTimeout: 2000,
-    rethrowAfterCapture: true,
     callbackWaitsForEmptyEventLoop: false,
     captureTimeoutWarning: true,
     timeoutWarningLimit: 500,
@@ -293,7 +291,7 @@ export function wrapHandler<TEvent, TResult>(
 
     const hub = getCurrentHub();
     const scope = hub.pushScope();
-    let rv: TResult | undefined;
+    let rv: TResult;
     try {
       enhanceScopeWithEnvironmentData(scope, context, START_TIME);
       // We put the transaction on the scope so users can attach children to it
@@ -309,9 +307,7 @@ export function wrapHandler<TEvent, TResult>(
       }
     } catch (e) {
       captureException(e);
-      if (options.rethrowAfterCapture) {
-        throw e;
-      }
+      throw e;
     } finally {
       clearTimeout(timeoutWarningTimer);
       transaction.finish();

packages/serverless/test/awslambda.test.ts

@@ -91,21 +91,6 @@ describe('AWSLambda', () => {
       expect(Sentry.flush).toBeCalledWith(1337);
     });
 
-    test('rethrowAfterCapture', async () => {
-      expect.assertions(3);
-
-      const error = new Error('wat');
-      const handler = () => {
-        throw error;
-      };
-      const wrappedHandlerWithRethrow = wrapHandler(handler, { rethrowAfterCapture: true });
-      const wrappedHandlerWithoutRethrow = wrapHandler(handler, { rethrowAfterCapture: false });
-
-      await expect(wrappedHandlerWithRethrow(fakeEvent, fakeContext, fakeCallback)).rejects.toThrow(error);
-      await expect(wrappedHandlerWithoutRethrow(fakeEvent, fakeContext, fakeCallback)).resolves.not.toThrow();
-      expect(Sentry.flush).toBeCalledTimes(2);
-    });
-
     test('captureTimeoutWarning enabled (default)', async () => {
       expect.assertions(2);