Merge pull request #44 from wemogy:fix/cors-order

Fix order of adding CORS
wemogy · Oct 20, 2023 · e7437cc · e7437cc
2 parents 9cac29f + 85db5b9
commit e7437cc
Showing 1 changed file with 4 additions and 2 deletions.
diff --git a/src/Wemogy.AspNet/Startup/StartupExtensions.cs b/src/Wemogy.AspNet/Startup/StartupExtensions.cs
@@ -86,6 +86,10 @@ public static void UseDefaultSetup(this IApplicationBuilder applicationBuilder,
                 applicationBuilder.UseDeveloperExceptionPage();
             }
 
+            // Must come before any "UseSwagger()" calls because the Swagger middleware, when it knows that the request
+            // is for Swagger, it doesn't forward the request onto the next middleware, it just immediately returns.
+            applicationBuilder.UseDefaultCors();
+
             if (options.OpenApiEnvironment != null)
             {
                 applicationBuilder.UseDefaultSwagger(options.OpenApiEnvironment);
@@ -103,8 +107,6 @@ public static void UseDefaultSetup(this IApplicationBuilder applicationBuilder,
                 applicationBuilder.UseCloudEvents();
             }
 
-            applicationBuilder.UseDefaultCors();
-
             applicationBuilder.UseAuthentication();
             applicationBuilder.UseAuthorization();