Fix for #111 and #116 (#118)

Fix for #111 (500 error for non-API endpoints) and #116 (PreprocessingDelegatingHandler breaks Swagger UI).

Saule/Http/PreprocessingDelegatingHandler.cs

@@ -11,6 +11,8 @@
 
 namespace Saule.Http
 {
+    using System.Linq;
+
     /// <summary>
     /// Processes JSON API responses to enable filtering, pagination and sorting.
     /// </summary>
@@ -61,10 +63,12 @@ internal static PreprocessResult PreprocessRequest(
         protected override async Task<HttpResponseMessage> SendAsync(HttpRequestMessage request, CancellationToken cancellationToken)
         {
             var result = await base.SendAsync(request, cancellationToken);
+            var hasMediaType = request.Headers.Accept.Any(x => x.MediaType == Constants.MediaType);
 
             var statusCode = (int)result.StatusCode;
-            if (statusCode >= 400 && statusCode < 500)
-            { // probably malformed request or not found
+            if (!hasMediaType || (statusCode >= 400 && statusCode < 500))
+            {
+                // probably malformed request or not found
                 return result;
             }
 

Tests/Controllers/StaticController.cs

@@ -0,0 +1,16 @@
+namespace Tests.Controllers
+{
+    using System.Net;
+    using System.Web.Http;
+
+    [RoutePrefix("static")]
+    public class StaticController : ApiController
+    {
+        [HttpGet]
+        [Route("text")]
+        public IHttpActionResult GetStaticFile()
+        {
+            return this.Content(HttpStatusCode.OK, "This is static content.");
+        }
+    }
+}

Tests/Helpers/Paths.cs

@@ -2,6 +2,7 @@
 {
     public static class Paths
     {
+        public const string StaticText = "static/text";
         public const string SingleResource = "api/people/123/";
         public const string ResourceCollection = "api/people/";
         public const string NonExistingPath = "api/wrong/";

Tests/Integration/ContentNegotiationTests.cs

@@ -162,6 +162,19 @@ public async Task MustReturn200OkForOneValidAccept()
 
                 Assert.Equal(HttpStatusCode.OK, result.StatusCode);
             }
+
+
+            [Fact(DisplayName = "Should return OK for a static content request that does not have media type parameters")]
+            public async Task MustReturn200OkForStaticContent()
+            {
+                var target = _server.GetClient();
+
+                target.DefaultRequestHeaders.Accept.Clear();
+
+                var result = await target.GetAsync(Paths.StaticText);
+
+                Assert.Equal(HttpStatusCode.OK, result.StatusCode);
+            }
         }
     }
 }

Tests/Tests.csproj

@@ -106,6 +106,7 @@
     <Compile Include="ApiResourceTests.cs" />
     <Compile Include="Controllers\BrokenController.cs" />
     <Compile Include="Controllers\CompaniesController.cs" />
+    <Compile Include="Controllers\StaticController.cs" />
     <Compile Include="Helpers\Get.cs" />
     <Compile Include="Helpers\HttpClientExtensions.cs" />
     <Compile Include="Helpers\NewSetupJsonApiServer.cs" />