GH-577 Ensure listeners are notified after dispatch

olegz · olegz · commit 071f7d378c14 · 2023-06-14T14:26:34.000+02:00
Resolves #577
diff --git a/aws-serverless-java-container-core/src/main/java/com/amazonaws/serverless/proxy/internal/servlet/AwsAsyncContext.java b/aws-serverless-java-container-core/src/main/java/com/amazonaws/serverless/proxy/internal/servlet/AwsAsyncContext.java
@@ -74,8 +74,8 @@ public void dispatch() {
                 throw new IllegalStateException("Dispatching already started");
             }
             dispatched.set(true);
-            notifyListeners(NotificationType.START_ASYNC, null);
             handler.doFilter(req, res, ((AwsServletContext)req.getServletContext()).getServletForPath(req.getRequestURI()));
+            notifyListeners(NotificationType.START_ASYNC, null);
         } catch (ServletException | IOException e) {
             notifyListeners(NotificationType.ERROR, e);
         }

Original file line number	Diff line number	Diff line change
`@@ -74,8 +74,8 @@ public void dispatch() {`
`74`	`74`	`throw new IllegalStateException("Dispatching already started");`
`75`	`75`	`}`
`76`	`76`	`dispatched.set(true);`
`77`		`- notifyListeners(NotificationType.START_ASYNC, null);`
`78`	`77`	`handler.doFilter(req, res, ((AwsServletContext)req.getServletContext()).getServletForPath(req.getRequestURI()));`
	`78`	`+ notifyListeners(NotificationType.START_ASYNC, null);`
`79`	`79`	`} catch (ServletException \| IOException e) {`
`80`	`80`	`notifyListeners(NotificationType.ERROR, e);`
`81`	`81`	`}`