Remove authorization for healthcheck API (#2105)

* remove authorization for healthcheck * use allow anonymous instead --------- Co-authored-by: chuongnguyen <[email protected]>
Azure · Dec 5, 2024 · 045aac4 · 045aac4
1 parent af7141a
commit 045aac4
Show file tree

Hide file tree

Showing 2 changed files with 10 additions and 0 deletions.
diff --git a/src/Microsoft.Azure.SignalR.Emulator/Common/SignalRServiceWebApiDefinition.cs b/src/Microsoft.Azure.SignalR.Emulator/Common/SignalRServiceWebApiDefinition.cs
@@ -28,6 +28,7 @@ internal abstract class SignalRServiceWebApiDefinition : ControllerBase
         /// <response code="200">The service is healthy</response>
         [HttpGet("health"), HttpHead("health")]
         [ProducesResponseType(200)]
+        [AllowAnonymous]
         public abstract IActionResult GetHealthStatus();
 
         // POST /api/v1/hubs/chat

diff --git a/test/Microsoft.Azure.SignalR.Emulator.Tests/Controllers/SignalRServiceEmulatorWebApiTests.cs b/test/Microsoft.Azure.SignalR.Emulator.Tests/Controllers/SignalRServiceEmulatorWebApiTests.cs
@@ -51,6 +51,15 @@ public SignalRServiceEmulatorWebApiTests()
             _controller.ControllerContext = controllerContext;
         }
 
+        [Fact]
+        public void HealthCheckReturnsOK()
+        {
+            var result = _controller.GetHealthStatus();
+
+            // assert
+            Assert.IsType<OkResult>(result);
+        }
+
         // CheckConnectionExistence Tests
         [Fact]
         public void CheckConnectionExistenceValidConnectionReturnsOk()