use ApiDescriptor for prefixes

Author: ibodrov

server/impl/src/main/java/com/walmartlabs/concord/server/boot/HttpServer.java

@@ -20,7 +20,7 @@
  * =====
  */
 
-import com.walmartlabs.concord.server.boot.resteasy.ApiDescriptor;
+import com.walmartlabs.concord.server.sdk.rest.ApiDescriptor;
 import com.walmartlabs.concord.server.cfg.ServerConfiguration;
 import org.eclipse.jetty.ee8.nested.SessionHandler;
 import org.eclipse.jetty.ee8.servlet.FilterHolder;

server/impl/src/main/java/com/walmartlabs/concord/server/boot/resteasy/ConcordApiDescriptor.java

@@ -20,6 +20,8 @@
  * =====
  */
 
+import com.walmartlabs.concord.server.sdk.rest.ApiDescriptor;
+
 public class ConcordApiDescriptor implements ApiDescriptor {
 
     @Override

server/impl/src/main/java/com/walmartlabs/concord/server/boot/resteasy/ResteasyModule.java

@@ -22,6 +22,7 @@
 
 import com.google.inject.Binder;
 import com.google.inject.Module;
+import com.walmartlabs.concord.server.sdk.rest.ApiDescriptor;
 import com.walmartlabs.concord.server.sdk.rest.Component;
 
 import javax.servlet.ServletContextListener;

server/plugins/webapp/pom.xml

@@ -16,6 +16,11 @@
     <packaging>jar</packaging>
 
     <dependencies>
+        <dependency>
+            <groupId>com.walmartlabs.concord.server</groupId>
+            <artifactId>concord-server-sdk</artifactId>
+            <scope>provided</scope>
+        </dependency>
         <dependency>
             <groupId>javax.inject</groupId>
             <artifactId>javax.inject</artifactId>

server/plugins/webapp/src/main/java/ca/ibodrov/concord/webapp/WebappFilter.java

@@ -9,9 +9,9 @@
  * Licensed under the Apache License, Version 2.0 (the "License");
  * you may not use this file except in compliance with the License.
  * You may obtain a copy of the License at
- * 
+ *
  *      http://www.apache.org/licenses/LICENSE-2.0
- * 
+ *
  * Unless required by applicable law or agreed to in writing, software
  * distributed under the License is distributed on an "AS IS" BASIS,
  * WITHOUT WARRANTIES OR CONDITIONS OF ANY KIND, either express or implied.
@@ -24,8 +24,10 @@
 import com.google.common.cache.CacheLoader;
 import com.google.common.cache.LoadingCache;
 import com.google.common.collect.ImmutableMap;
+import com.walmartlabs.concord.server.sdk.rest.ApiDescriptor;
 
 import javax.annotation.Priority;
+import javax.inject.Inject;
 import javax.servlet.FilterChain;
 import javax.servlet.ServletException;
 import javax.servlet.annotation.WebFilter;
@@ -41,6 +43,8 @@
 import java.util.stream.Stream;
 
 import static java.util.Comparator.comparing;
+import static java.util.Comparator.reverseOrder;
+import static java.util.Objects.requireNonNull;
 import static javax.ws.rs.core.Response.Status.INTERNAL_SERVER_ERROR;
 
 /**
@@ -51,17 +55,29 @@
 public class WebappFilter extends HttpFilter {
 
     private final WebappCollection webapps;
+    private final List<String> apiPrefixes;
 
-    public WebappFilter() {
+    @Inject
+    public WebappFilter(Set<ApiDescriptor> apiDescriptors) {
         this.webapps = loadWebapps();
+        this.apiPrefixes = requireNonNull(apiDescriptors).stream()
+                .flatMap(descriptor -> Arrays.stream(descriptor.paths()))
+                .map(prefix -> {
+                    if (prefix.endsWith("/*")) {
+                        return prefix.substring(0, prefix.length() - 1);
+                    }
+                    return prefix;
+                })
+                .sorted(reverseOrder())
+                .toList();
     }
 
     @Override
     protected void doFilter(HttpServletRequest req, HttpServletResponse resp, FilterChain chain)
             throws ServletException, IOException {
 
         var uri = req.getRequestURI();
-        if (uri.startsWith("/api")) {
+        if (apiPrefixes.stream().anyMatch(uri::startsWith)) {
             chain.doFilter(req, resp);
             return;
         }

server/plugins/webapp/src/main/java/ca/ibodrov/concord/webapp/WebappPluginModule.java

@@ -22,6 +22,7 @@
 
 import com.google.inject.Binder;
 import com.google.inject.Module;
+import com.walmartlabs.concord.server.sdk.rest.ApiDescriptor;
 
 import javax.inject.Named;
 import javax.servlet.Filter;
@@ -33,6 +34,7 @@ public class WebappPluginModule implements Module {
 
     @Override
     public void configure(Binder binder) {
+        newSetBinder(binder, ApiDescriptor.class);
         newSetBinder(binder, Filter.class).addBinding().to(WebappFilter.class);
     }
 }

server/impl/src/main/java/com/walmartlabs/concord/server/boot/resteasy/ApiDescriptor.java renamed to server/sdk/src/main/java/com/walmartlabs/concord/server/sdk/rest/ApiDescriptor.java

@@ -1,4 +1,4 @@
-package com.walmartlabs.concord.server.boot.resteasy;
+package com.walmartlabs.concord.server.sdk.rest;
 
 /*-
  * *****
@@ -21,7 +21,7 @@
  */
 
 /**
- * Declares an API.
+ * Declares a list of prefixes that should be served as JAX-RS resources.
  */
 public interface ApiDescriptor {