concord-server: allow custom top-level API endpoints

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

@@ -20,6 +20,7 @@
  * =====
  */
 
+import com.walmartlabs.concord.server.boot.resteasy.ApiDescriptor;
 import com.walmartlabs.concord.server.cfg.ServerConfiguration;
 import org.eclipse.jetty.ee8.nested.SessionHandler;
 import org.eclipse.jetty.ee8.servlet.FilterHolder;
@@ -31,6 +32,7 @@
 import org.eclipse.jetty.server.*;
 import org.eclipse.jetty.server.handler.ContextHandlerCollection;
 import org.eclipse.jetty.server.handler.StatisticsHandler;
+import org.jboss.resteasy.plugins.server.servlet.HttpServletDispatcher;
 import org.slf4j.Logger;
 import org.slf4j.LoggerFactory;
 
@@ -56,6 +58,7 @@ public class HttpServer {
     public HttpServer(ServerConfiguration cfg,
                       Set<RequestErrorHandler> requestErrorHandlers,
                       Set<ServletContextListener> contextListeners,
+                      Set<ApiDescriptor> apiDescriptors,
                       Set<HttpServlet> servlets,
                       Set<ServletHolder> servletHolders,
                       Set<Filter> filters,
@@ -119,6 +122,15 @@ public HttpServer(ServerConfiguration cfg,
             contextHandler.addEventListener(listener);
         });
 
+        // init all Resteasy endpoints
+        apiDescriptors.forEach(api -> {
+            ServletHolder holder = new ServletHolder(HttpServletDispatcher.class);
+            for (String pathSpec : api.paths()) {
+                log.info("Serving API endpoints @ {}", pathSpec);
+                contextHandler.addServlet(holder, pathSpec);
+            }
+        });
+
         // init all @WebServlets
         servlets.stream().sorted(byPriority()).forEachOrdered(servlet -> {
             WebServlet annotation = servlet.getClass().getAnnotation(WebServlet.class);

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

@@ -0,0 +1,32 @@
+package com.walmartlabs.concord.server.boot.resteasy;
+
+/*-
+ * *****
+ * Concord
+ * -----
+ * Copyright (C) 2017 - 2024 Walmart Inc.
+ * -----
+ * 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.
+ * See the License for the specific language governing permissions and
+ * limitations under the License.
+ * =====
+ */
+
+/**
+ * Declares an API.
+ */
+public interface ApiDescriptor {
+
+    /**
+     * Path patterns of the API. E.g. {@code "/api/**"} or {@code "/events/foo"}.
+     */
+    String[] paths();
+}

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

@@ -4,7 +4,7 @@
  * *****
  * Concord
  * -----
- * Copyright (C) 2017 - 2023 Walmart Inc.
+ * Copyright (C) 2017 - 2024 Walmart Inc.
  * -----
  * Licensed under the Apache License, Version 2.0 (the "License");
  * you may not use this file except in compliance with the License.
@@ -20,18 +20,13 @@
  * =====
  */
 
-import org.eclipse.jetty.ee8.servlet.ServletHolder;
-import org.jboss.resteasy.plugins.server.servlet.HttpServletDispatcher;
+public class ConcordApiDescriptor implements ApiDescriptor {
 
-import javax.servlet.annotation.WebServlet;
-
-@WebServlet({
-        "/api/*",
-        "/events/github/*"
-})
-public class ResteasyServletHolder extends ServletHolder {
-
-    public ResteasyServletHolder() {
-        super(HttpServletDispatcher.class);
+    @Override
+    public String[] paths() {
+        return new String[]{
+                "/api/*",
+                "/events/github/*"
+        };
     }
 }

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

@@ -29,13 +29,12 @@
 import static com.google.inject.Scopes.SINGLETON;
 import static com.google.inject.multibindings.Multibinder.newSetBinder;
 import static com.walmartlabs.concord.server.Utils.bindExceptionMapper;
-import static com.walmartlabs.concord.server.Utils.bindServletHolder;
 
 public class ResteasyModule implements Module {
 
     @Override
     public void configure(Binder binder) {
-        bindServletHolder(binder, ResteasyServletHolder.class);
+        newSetBinder(binder, ApiDescriptor.class).addBinding().to(ConcordApiDescriptor.class);
 
         newSetBinder(binder, ServletContextListener.class).addBinding().to(ResteasyListener.class);