Simplified route config

samples/IslandGateway.Sample/appsettings.json

@@ -34,7 +34,9 @@
         {
           "RouteId": "backend1/route1",
           "BackendId": "backend1",
-          "Rule": "Host('localhost') && Path('/{**catchall}')"
+          "Methods": [ "GET", "POST" ],
+          "Host": "localhost",
+          "Path": "/{**catchall}"
         }
       ]
     }

src/IslandGateway.Core/Abstractions/RouteDiscovery/Contract/GatewayRoute.cs

@@ -1,4 +1,4 @@
-// Copyright (c) Microsoft Corporation.
+// Copyright (c) Microsoft Corporation.
 // Licensed under the MIT License.
 
 using System;
@@ -7,7 +7,7 @@
 namespace IslandGateway.Core.Abstractions
 {
     /// <summary>
-    /// Describes a route that matches incoming requests based on a <see cref="Rule"/>
+    /// Describes a route that matches incoming requests based on a <see cref="Methods"/>, <see cref="Host"/>, and <see cref="Path"/>
     /// and proxies matching requests to the backend identified by its <see cref="BackendId"/>.
     /// </summary>
     public sealed class GatewayRoute : IDeepCloneable<GatewayRoute>
@@ -18,9 +18,31 @@ public sealed class GatewayRoute : IDeepCloneable<GatewayRoute>
         public string RouteId { get; set; }
 
         /// <summary>
-        /// Rule that incoming requests must match for this route to apply. E.g. <c>Host('example.com')</c>.
+        /// Only match requests that use these optional HTTP methods. E.g. GET, POST.
         /// </summary>
-        public string Rule { get; set; }
+        public string[] Methods { get; set; }
+
+        /// <summary>
+        /// Only match requests with the given Host header.
+        /// </summary>
+        public string Host { get; set; }
+
+        /// <summary>
+        /// Only match requests with the given Path pattern.
+        /// </summary>
+        public string Path { get; set; }
+
+        // TODO:
+        /// <summary>
+        /// Only match requests that contain all of these query parameters.
+        /// </summary>
+        // public ICollection<KeyValuePair<string, string>> QueryParameters { get; set; }
+
+        // TODO:
+        /// <summary>
+        /// Only match requests that contain all of these request headers.
+        /// </summary>
+        // public ICollection<KeyValuePair<string, string>> Headers { get; set; }
 
         /// <summary>
         /// Optionally, a priority value for this route. Routes with higher numbers take precedence over lower numbers.
@@ -44,7 +66,10 @@ GatewayRoute IDeepCloneable<GatewayRoute>.DeepClone()
             return new GatewayRoute
             {
                 RouteId = RouteId,
-                Rule = Rule,
+                Methods = (string[])Methods?.Clone(),
+                Host = Host,
+                Path = Path,
+                // Headers = Headers.DeepClone(); // TODO:
                 Priority = Priority,
                 BackendId = BackendId,
                 Metadata = Metadata?.DeepClone(StringComparer.Ordinal),

src/IslandGateway.Core/Configuration/DependencyInjection/BuilderExtensions/Core.cs

@@ -1,4 +1,4 @@
-// Copyright (c) Microsoft Corporation.
+// Copyright (c) Microsoft Corporation.
 // Licensed under the MIT License.
 
 using IslandGateway.Common.Abstractions.Telemetry;
@@ -43,8 +43,6 @@ public static IIslandGatewayBuilder AddInMemoryRepos(this IIslandGatewayBuilder
         public static IIslandGatewayBuilder AddConfigBuilder(this IIslandGatewayBuilder builder)
         {
             builder.Services.AddSingleton<IDynamicConfigBuilder, DynamicConfigBuilder>();
-            builder.Services.AddSingleton<IRuleParser, RuleParser>();
-            builder.Services.AddSingleton<IRouteParser, RouteParser>();
             builder.Services.AddSingleton<IRouteValidator, RouteValidator>();
             builder.Services.AddSingleton<IRuntimeRouteBuilder, RuntimeRouteBuilder>();
             return builder;

src/IslandGateway.Core/IslandGateway.Core.csproj

@@ -10,10 +10,6 @@
     <FrameworkReference Include="Microsoft.AspNetCore.App" />
   </ItemGroup>
 
-  <ItemGroup>
-    <PackageReference Include="Superpower" Version="2.3.0" />
-  </ItemGroup>
-
   <ItemGroup>
     <ProjectReference Include="..\IslandGateway.Common\IslandGateway.Common.csproj" />
     <ProjectReference Include="..\IslandGateway.Utilities\IslandGateway.Utilities.csproj" />

src/IslandGateway.Core/Service/Config/ConfigErrors.cs

@@ -1,4 +1,4 @@
-// Copyright (c) Microsoft Corporation.
+// Copyright (c) Microsoft Corporation.
 // Licensed under the MIT License.
 
 namespace IslandGateway.Core.Service
@@ -14,7 +14,6 @@ internal static class ConfigErrors
         internal const string RouteUnknownBackend = "Route_UnknownBackend";
         internal const string RouteNoBackends = "Route_NoBackends";
         internal const string RouteUnsupportedAction = "Route_UnsupportedAction";
-        internal const string RouteBadRule = "Route_BadRule";
 
         internal const string ParsedRouteMissingId = "ParsedRoute_MissingId";
         internal const string ParsedRouteRuleHasNoMatchers = "ParsedRoute_RuleHasNoMatchers";

src/IslandGateway.Core/Service/Config/DynamicConfigBuilder.cs

@@ -1,4 +1,4 @@
-// Copyright (c) Microsoft Corporation.
+// Copyright (c) Microsoft Corporation.
 // Licensed under the MIT License.
 
 using System;
@@ -16,26 +16,22 @@ internal class DynamicConfigBuilder : IDynamicConfigBuilder
         private readonly IBackendsRepo _backendsRepo;
         private readonly IBackendEndpointsRepo _endpointsRepo;
         private readonly IRoutesRepo _routesRepo;
-        private readonly IRouteParser _routeParser;
         private readonly IRouteValidator _parsedRouteValidator;
 
         public DynamicConfigBuilder(
             IBackendsRepo backendsRepo,
             IBackendEndpointsRepo endpointsRepo,
             IRoutesRepo routesRepo,
-            IRouteParser routeParser,
             IRouteValidator parsedRouteValidator)
         {
             Contracts.CheckValue(backendsRepo, nameof(backendsRepo));
             Contracts.CheckValue(endpointsRepo, nameof(endpointsRepo));
             Contracts.CheckValue(routesRepo, nameof(routesRepo));
-            Contracts.CheckValue(routeParser, nameof(routeParser));
             Contracts.CheckValue(parsedRouteValidator, nameof(parsedRouteValidator));
 
             _backendsRepo = backendsRepo;
             _endpointsRepo = endpointsRepo;
             _routesRepo = routesRepo;
-            _routeParser = routeParser;
             _parsedRouteValidator = parsedRouteValidator;
         }
 
@@ -118,14 +114,16 @@ private async Task<IList<ParsedRoute>> GetRoutesAsync(IConfigErrorReporter error
                         continue;
                     }
 
-                    var parsedResult = _routeParser.ParseRoute(route, errorReporter);
-                    if (!parsedResult.IsSuccess)
-                    {
-                        // routeParser already reported error message
-                        continue;
-                    }
+                    var parsedRoute = new ParsedRoute {
+                        RouteId = route.RouteId,
+                        Methods = route.Methods,
+                        Host = route.Host,
+                        Path = route.Path,
+                        Priority = route.Priority,
+                        BackendId = route.BackendId,
+                        Metadata = route.Metadata,
+                    };
 
-                    var parsedRoute = parsedResult.Value;
                     if (!_parsedRouteValidator.ValidateRoute(parsedRoute, errorReporter))
                     {
                         // parsedRouteValidator already reported error message