Revert RoutingFilter use of try-with-resources

avaje · Nov 26, 2024 · aa9af57 · aa9af57
1 parent 7050aa3
commit aa9af57
Showing 1 changed file with 4 additions and 2 deletions.
diff --git a/avaje-jex/src/main/java/io/avaje/jex/jdk/RoutingFilter.java b/avaje-jex/src/main/java/io/avaje/jex/jdk/RoutingFilter.java
@@ -37,7 +37,7 @@ public void doFilter(HttpExchange exchange, Filter.Chain chain) {
     if (route == null) {
       var ctx = new JdkContext(mgr, exchange, uri, Set.of());
       routes.inc();
-      try (exchange) {
+      try {
         processNoRoute(ctx, uri, routeType);
         exchange.setAttribute("JdkContext", ctx);
         chain.doFilter(exchange);
@@ -46,10 +46,11 @@ public void doFilter(HttpExchange exchange, Filter.Chain chain) {
         handleException(ctx, e);
       } finally {
         routes.dec();
+        exchange.close();
       }
     } else {
       route.inc();
-      try (exchange) {
+      try {
         final Map<String, String> params = route.pathParams(uri);
         JdkContext ctx = new JdkContext(mgr, exchange, route.matchPath(), params, route.roles());
         try {
@@ -64,6 +65,7 @@ public void doFilter(HttpExchange exchange, Filter.Chain chain) {
         }
       } finally {
         route.dec();
+        exchange.close();
       }
     }
   }