Fix warnings

ThreeMammals · Aug 25, 2023 · 3560bb9 · 3560bb9
1 parent 90a5f3e
commit 3560bb9
Show file tree

Hide file tree

Showing 2 changed files with 16 additions and 21 deletions.
diff --git a/src/Ocelot/Middleware/OcelotPipelineConfiguration.cs b/src/Ocelot/Middleware/OcelotPipelineConfiguration.cs
@@ -36,18 +36,15 @@ public class OcelotPipelineConfiguration
         public Func<HttpContext, Func<Task>, Task> AuthenticationMiddleware { get; set; }
 
         /// <summary>
-        /// This is to allow the user to run any extra authentication after the Ocelot authentication
-        /// kicks in
+        /// This is to allow the user to run any extra authentication after the Ocelot authentication kicks in.
         /// </summary>
         /// <value>
-        /// <placeholder>This is to allow the user to run any extra authentication after the Ocelot authentication
-        /// kicks in</placeholder>
+        /// A <see cref="Func{HttpContext, TFunc, Task}"/> delegate object.
         /// </value>
         public Func<HttpContext, Func<Task>, Task> AfterAuthenticationMiddleware { get; set; }
 
         /// <summary>
-        /// This is to allow the user to run any extra authorization before the Ocelot authentication
-        /// kicks in
+        /// This is to allow the user to run any extra authorization before the Ocelot authentication kicks in.
         /// </summary>
         /// <value>
         /// A <see cref="Func{HttpContext, TFunc, Task}"/> delegate object.
@@ -63,17 +60,15 @@ public class OcelotPipelineConfiguration
         public Func<HttpContext, Func<Task>, Task> AuthorizationMiddleware { get; set; }
 
         /// <summary>
-        /// This is to allow the user to run any extra authorization after the Ocelot authentication
-        /// kicks in
+        /// This is to allow the user to run any extra authorization after the Ocelot authorization kicks in.
         /// </summary>
         /// <value>
-        /// <placeholder>This is to allow the user to run any extra authorization after the Ocelot authentication
-        /// kicks in</placeholder>
+        /// A <see cref="Func{HttpContext, TFunc, Task}"/> delegate object.
         /// </value>
         public Func<HttpContext, Func<Task>, Task> AfterAuthorizationMiddleware { get; set; }
 
         /// <summary>
-        /// This allows the user to implement there own query string manipulation logic
+        /// This allows the user to implement there own query string manipulation logic.
         /// </summary>
         /// <value>
         /// A <see cref="Func{HttpContext, TFunc, Task}"/> delegate object.

diff --git a/test/Ocelot.AcceptanceTests/CustomMiddlewareTests.cs b/test/Ocelot.AcceptanceTests/CustomMiddlewareTests.cs
@@ -263,11 +263,11 @@ public void should_call_after_authorization_middleware()
         {
             var configuration = new OcelotPipelineConfiguration
             {
-                AfterAuthorizationMiddleware =  async (ctx, next) =>
+                AfterAuthorizationMiddleware = async (ctx, next) =>
                 {
                     _counter++;
                     await next.Invoke();
-                }
+                },
             };
 
             var port = RandomPortFinder.GetRandomPort();
@@ -285,13 +285,13 @@ public void should_call_after_authorization_middleware()
                             {
                                 Host = "localhost",
                                 Port = port,
-                            }
+                            },
                         },
                         DownstreamScheme = "http",
                         UpstreamPathTemplate = "/",
                         UpstreamHttpMethod = new List<string> { "Get" },
-                    }
-                }
+                    },
+                },
             };
 
             this.Given(x => x.GivenThereIsAServiceRunningOn($"http://localhost:{port}", 200, ""))
@@ -353,11 +353,11 @@ public void should_call_after_http_authentication_middleware()
         {
             var configuration = new OcelotPipelineConfiguration
             {
-                AfterAuthenticationMiddleware =  async (ctx, next) =>
+                AfterAuthenticationMiddleware = async (ctx, next) =>
                 {
                     _counter++;
                     await next.Invoke();
-                }
+                },
             };
 
             var port = RandomPortFinder.GetRandomPort();
@@ -375,13 +375,13 @@ public void should_call_after_http_authentication_middleware()
                             {
                                 Host = "localhost",
                                 Port = port,
-                            }
+                            },
                         },
                         DownstreamScheme = "http",
                         UpstreamPathTemplate = "/",
                         UpstreamHttpMethod = new List<string> { "Get" },
-                    }
-                }
+                    },
+                },
             };
 
             this.Given(x => x.GivenThereIsAServiceRunningOn($"http://localhost:{port}", 200, ""))