Fix 400 returned when server is not found (#98)

ArmaForces · Sep 14, 2024 · 1f225a0 · 1f225a0
1 parent aacded7
commit 1f225a0
Showing 1 changed file with 3 additions and 2 deletions.
diff --git a/ArmaForces.ArmaServerManager/Api/Servers/ServersController.cs b/ArmaForces.ArmaServerManager/Api/Servers/ServersController.cs
@@ -118,7 +118,7 @@ public async Task<IActionResult> SetHeadlessClients(int port, HeadlessSetRequest
 
             return result.Match(
                 onSuccess: NoContent,
-                onFailure: error => (IActionResult) BadRequest(error));
+                onFailure: error => (IActionResult) NotFound(error));
         }
 
         /// <summary>Restart Server</summary>
@@ -127,14 +127,15 @@ public async Task<IActionResult> SetHeadlessClients(int port, HeadlessSetRequest
         /// <param name="serverRestartRequestDto">Additional details.</param>
         [HttpPost("{port:int}/restart", Name = nameof(RestartServer))]
         [ProducesResponseType(typeof(int), StatusCodes.Status202Accepted)]
+        [ProducesResponseType(typeof(string), StatusCodes.Status404NotFound)]
         [ProducesResponseType(typeof(string), StatusCodesExtended.Status425TooEarly)]
         [ProducesResponseType(typeof(string), StatusCodes.Status500InternalServerError)]
         [ApiKey]
         public IActionResult RestartServer(int port, ServerRestartRequestDto serverRestartRequestDto)
         {
             var serverGetResult = _serverQueryLogic.GetServer(port);
 
-            if (serverGetResult.IsFailure) return BadRequest("Server is not running and cannot be restarted.");
+            if (serverGetResult.IsFailure) return NotFound("Server is not running and cannot be restarted.");
 
             var server = serverGetResult.Value;