[GitHub] [jena] kinow commented on a change in pull request #1198: JENA-2281: Routing all request variations of request URIs.

[GitBox <gi...@apache.org>]

kinow commented on a change in pull request #1198:
URL: https://github.com/apache/jena/pull/1198#discussion_r807282128



##########
File path: jena-fuseki2/jena-fuseki-core/src/main/java/org/apache/jena/fuseki/server/Dispatcher.java
##########
@@ -73,27 +73,73 @@
      * This function does not throw exceptions.
      */
     public static boolean dispatch(HttpServletRequest request, HttpServletResponse response) {
+        DataAccessPointRegistry registry = DataAccessPointRegistry.get(request.getServletContext());
+        DataAccessPoint dap = locateDataAccessPoint(request, registry);
+        if ( dap == null ) {
+            if ( LogDispatch )
+                LOG.debug("No dispatch for '"+request.getRequestURI()+"'");
+            return false;
+        }
+        return process(dap, request, response);
+    }
+
+    /**
+     * The request may be /path/dataset/sparql or /path/dataset, or even /.
+     * <p>
+     * If the servlet context path is the "/path", then that was handled in  ActionLib.actionURI.

Review comment:
       extra space before ActionLib.actionURI




-- 
This is an automated message from the Apache Git Service.
To respond to the message, please log on to GitHub and use the
URL above to go to the specific comment.

To unsubscribe, e-mail: pr-unsubscribe@jena.apache.org

For queries about this service, please contact Infrastructure at:
users@infra.apache.org



---------------------------------------------------------------------
To unsubscribe, e-mail: pr-unsubscribe@jena.apache.org
For additional commands, e-mail: pr-help@jena.apache.org