feat: Enhance LocalPortFiltering to support multiple allowed ports.

- Updated the `AllowPort` property to `List<int> AllowPorts` to handle multiple allowed ports.
- Updated documentation and comments accordingly.

Author: willysoft

README.md

@@ -33,7 +33,7 @@ app.UseLocalPortFiltering();
 
 // Define a GET endpoint with port filtering
 app.MapGet("/service1", () => "Welcome to Service 1")
-    .RequireLocalPortFiltering(allowPort: 5105);
+   .RequireLocalPortFiltering(allowPorts: 5105);
 
 // Define another GET endpoint without port filtering
 app.MapGet("/service2", () => "Welcome to Service 2");
@@ -54,7 +54,7 @@ using Microsoft.AspNetCore.Mvc;
 public class SampleController : ControllerBase
 {
     [HttpGet]
-    [LocalPortFiltering(allowPort: 5105)]
+    [LocalPortFiltering(allowPorts: 5105)]
     public IActionResult Get()
     {
         return Ok("This endpoint is only accessible on port 5105.");
@@ -84,7 +84,7 @@ Use `RequireLocalPortFiltering`
 You can apply port-based filtering to specific endpoints:
 
 ```csharp
-app.MapHealthChecks("/healthz").RequireLocalPortFiltering(allowPort: 5105);
+app.MapHealthChecks("/healthz").RequireLocalPortFiltering(allowPorts: 5105);
 ```
 
 **Combine with Conditional Middleware**

sample/LocalPortFilteringDemo/Controllers/WeatherForecastController.cs

@@ -19,7 +19,7 @@ public WeatherForecastController(ILogger<WeatherForecastController> logger)
     }
 
     [HttpGet(Name = "GetWeatherForecast")]
-    [LocalPortFiltering(allowPort: 7239)]
+    [LocalPortFiltering(allowPorts: 7239)]
     public IEnumerable<WeatherForecast> Get()
     {
         return Enumerable.Range(1, 5).Select(index => new WeatherForecast

sample/LocalPortFilteringDemo/Program.cs

@@ -16,7 +16,7 @@
 app.UseAuthorization();
 app.UseLocalPortFiltering();
 
-app.MapHealthChecks("/healthz").RequireLocalPortFiltering(allowPort: 5105);
+app.MapHealthChecks("/healthz").RequireLocalPortFiltering(allowPorts: 5105);
 app.MapControllers();
 
 app.Run();

src/LocalPortFiltering.AspNetCore/ILocalPortFilteringData.cs

@@ -6,8 +6,8 @@
 public interface ILocalPortFilteringData
 {
     /// <summary>
-    /// Gets the port number that is allowed through the filter.
+    /// Gets the list of port numbers that are allowed through the filter.
     /// </summary>
-    int AllowPort { get; }
+    IReadOnlyList<int> AllowPorts { get; }
 }
 

src/LocalPortFiltering.AspNetCore/LocalPortFilteringAttribute.cs

@@ -5,14 +5,25 @@
 public class LocalPortFilteringAttribute : Attribute, ILocalPortFilteringData
 {
     /// <inheritdoc />
-    public int AllowPort { get; }
+    public IReadOnlyList<int> AllowPorts { get; }
 
     /// <summary>
     /// Initializes a new instance of the <see cref="LocalPortFilteringAttribute"/> class.
     /// </summary>
     /// <param name="allowPort">The port number that is allowed through the filter.</param>
     public LocalPortFilteringAttribute(int allowPort)
+         : this([allowPort])
     {
-        AllowPort = allowPort;
+    }
+
+    /// <summary>
+    /// Initializes a new instance of the <see cref="LocalPortFilteringAttribute"/> class.
+    /// </summary>
+    /// <param name="allowPorts">The port numbers that is allowed through the filter.</param>
+    public LocalPortFilteringAttribute(params int[] allowPorts)
+    {
+        ArgumentNullException.ThrowIfNull(allowPorts);
+
+        AllowPorts = allowPorts.ToArray();
     }
 }

src/LocalPortFiltering.AspNetCore/LocalPortFilteringEndpointConventionBuilderExtensions.cs

@@ -12,14 +12,14 @@ public static class LocalPortFilteringEndpointConventionBuilderExtensions
     /// </summary>
     /// <typeparam name="TBuilder">The type of the builder.</typeparam>
     /// <param name="builder">The endpoint convention builder instance.</param>
-    /// <param name="allowPort">The allowed port for local port filtering.</param>
+    /// <param name="allowPorts">The allowed ports for local port filtering.</param>
     /// <returns>The updated endpoint convention builder.</returns>
-    public static TBuilder RequireLocalPortFiltering<TBuilder>(this TBuilder builder, int allowPort)
+    public static TBuilder RequireLocalPortFiltering<TBuilder>(this TBuilder builder, params int[] allowPorts)
         where TBuilder : IEndpointConventionBuilder
     {
         ArgumentNullException.ThrowIfNull(builder);
 
-        RequireLocalPortFilteringCore(builder, new LocalPortFilteringAttribute(allowPort));
+        RequireLocalPortFilteringCore(builder, new LocalPortFilteringAttribute(allowPorts));
 
         return builder;
     }

src/LocalPortFiltering.AspNetCore/LocalPortFilteringMiddleware.cs

@@ -51,7 +51,7 @@ public Task Invoke(HttpContext context)
         if (exceptionHandler == null)
             return m_Next(context);
 
-        if (exceptionHandler.AllowPort != context.Connection.LocalPort)
+        if (!exceptionHandler.AllowPorts.Contains(context.Connection.LocalPort))
             return HostValidationFailed(context);
 
         return m_Next(context);

test/LocalPortFiltering.AspNetCore.Tests/LocalPortFilteringConventionBuilderExtensionsTests.cs

@@ -1,5 +1,4 @@
 using Microsoft.AspNetCore.Builder;
-using Microsoft.AspNetCore.Cors.Infrastructure;
 using Microsoft.AspNetCore.Http;
 
 namespace LocalPortFiltering.AspNetCore.Tests;
@@ -12,7 +11,7 @@ public void RequireLocalPort_AllowPort_MetadataAdded()
         var testConventionBuilder = new TestEndpointConventionBuilder();
 
         // Act
-        testConventionBuilder.RequireLocalPortFiltering(80);
+        testConventionBuilder.RequireLocalPortFiltering([80, 443]);
 
         // Assert
         var addCorsPolicy = Assert.Single(testConventionBuilder.Conventions);
@@ -23,7 +22,8 @@ public void RequireLocalPort_AllowPort_MetadataAdded()
 
         var metadata = endpoint.Metadata.GetMetadata<ILocalPortFilteringData>();
         Assert.NotNull(metadata);
-        Assert.Equal(80, metadata.AllowPort);
+        Assert.Contains(80, metadata.AllowPorts);
+        Assert.Contains(443, metadata.AllowPorts);
     }
 
     [Fact]

test/LocalPortFiltering.AspNetCore.Tests/LocalPortFilteringMiddlewareTests.cs

@@ -42,9 +42,13 @@ public async Task TestLocalPortFiltering_DefaultBehavior()
     }
 
     [Theory]
-    [InlineData(200, 80, 80)]
-    [InlineData(403, 8080, 80)]
-    public async Task TestLocalPortFiltering_RejectsBlockedPort(int status, int localPort, int allowPort)
+    [InlineData(200, 443, new int[] { 443 })]
+    [InlineData(403, 80, new int[] { 443 })]
+    [InlineData(200, 3000, new int[] { 3000, 3001 })]
+    [InlineData(403, 4000, new int[] { 3000, 3001 })]
+    [InlineData(200, 1005, new int[] { 1000, 1001, 1002, 1003, 1004, 1005, 1006, 1007, 1008, 1009, 1010 })]
+    [InlineData(403, 1500, new int[] { 1000, 1001, 1002, 1003, 1004, 1005, 1006, 1007, 1008, 1009, 1010 })]
+    public async Task TestLocalPortFiltering_RejectsBlockedPort(int status, int localPort, int[] allowPort)
     {
         // Arrange
         var options = new LocalPortFilteringOptions();