3redronin/murp

A reverse proxy handler for Mu Server

murp's Issues

[Suggestion] Handle WebApplicationException in requestInterceptor

With UriMapper one can check details (auth header for example) of the client request by examine the MuRequest argument, and throw WebApplicationException in uri mapper, but this sounds not very intuitive. To catch and re-throw WebApplicationException in RequestInterceptor looks better.

Index: src/main/java/io/muserver/murp/ReverseProxy.java
IDEA additional info:
Subsystem: com.intellij.openapi.diff.impl.patch.CharsetEP
<+>UTF-8
===================================================================
diff --git a/src/main/java/io/muserver/murp/ReverseProxy.java b/src/main/java/io/muserver/murp/ReverseProxy.java
--- a/src/main/java/io/muserver/murp/ReverseProxy.java	(revision df2abae1736303d2b6d617fdcfb02ac582e71235)
+++ b/src/main/java/io/muserver/murp/ReverseProxy.java	(date 1702447573220)
@@ -4,6 +4,7 @@
 import org.slf4j.Logger;
 import org.slf4j.LoggerFactory;
 
+import javax.ws.rs.WebApplicationException;
 import java.net.InetAddress;
 import java.net.URI;
 import java.net.http.HttpClient;
@@ -291,6 +292,9 @@
         if (requestInterceptor != null) {
             try {
                 requestInterceptor.intercept(clientRequest, targetReq);
+            } catch (WebApplicationException webEx) {
+                log.error("requestInterceptor error", webEx);
+                throw webEx;
             } catch (Throwable throwable) {
                 log.info("requestInterceptor error", throwable);
                 clientResponse.status(500);