Remove useless URL-rewriting logic from the Java FF. The code seems to be based on the idea that the function servlet is registered with a servlet path that is the name of the function (/myfunction) but actually the servlet path is always empty. So the existing logic has no effect.

PiperOrigin-RevId: 296277143

Author: eamonnmcmanus

invoker/core/src/main/java/com/google/cloud/functions/invoker/HttpFunctionExecutor.java

@@ -30,9 +30,8 @@ public HttpFunctionExecutor(HttpCloudFunction function) {
   /** Executes the user's method, can handle all HTTP type methods. */
   @Override
   public void service(HttpServletRequest req, HttpServletResponse res) {
-    URLRequestWrapper wrapper = new URLRequestWrapper(req);
     try {
-      function.execute(wrapper, res);
+      function.execute(req, res);
     } catch (InvocationTargetException e) {
       res.setStatus(HttpServletResponse.SC_INTERNAL_SERVER_ERROR);
       e.getCause().printStackTrace();

invoker/core/src/main/java/com/google/cloud/functions/invoker/NewHttpFunctionExecutor.java

@@ -59,8 +59,7 @@ public static NewHttpFunctionExecutor forClass(Class<?> functionClass) {
   /** Executes the user's method, can handle all HTTP type methods. */
   @Override
   public void service(HttpServletRequest req, HttpServletResponse res) {
-    URLRequestWrapper wrapper = new URLRequestWrapper(req);
-    HttpRequestImpl reqImpl = new HttpRequestImpl(wrapper);
+    HttpRequestImpl reqImpl = new HttpRequestImpl(req);
     HttpResponseImpl respImpl = new HttpResponseImpl(res);
     try {
       function.service(reqImpl, respImpl);