chore(controllers): enrich log events

Author: nanotaboada

src/Dotnet.Samples.AspNetCore.WebApi/Controllers/PlayerController.cs

@@ -41,17 +41,22 @@ public async Task<IResult> PostAsync([FromBody] PlayerRequestModel player)
                 .Errors.Select(error => new { error.PropertyName, error.ErrorMessage })
                 .ToArray();
 
-            logger.LogWarning("POST validation failed: {@Errors}", errors);
+            logger.LogWarning("POST /players validation failed: {@Errors}", errors);
             return TypedResults.BadRequest(errors);
         }
 
         if (await playerService.RetrieveByIdAsync(player.Id) != null)
         {
+            logger.LogWarning(
+                "POST /players failed: Player with ID {Id} already exists",
+                player.Id
+            );
             return TypedResults.Conflict();
         }
 
         var result = await playerService.CreateAsync(player);
 
+        logger.LogInformation("POST /players created: {@Player}", result);
         return TypedResults.CreatedAtRoute(
             routeName: "GetById",
             routeValues: new { id = result.Id },
@@ -77,10 +82,12 @@ public async Task<IResult> GetAsync()
 
         if (players.Count > 0)
         {
+            logger.LogInformation("GET /players retrieved");
             return TypedResults.Ok(players);
         }
         else
         {
+            logger.LogWarning("GET /players not found");
             return TypedResults.NotFound();
         }
     }
@@ -97,13 +104,14 @@ public async Task<IResult> GetAsync()
     public async Task<IResult> GetByIdAsync([FromRoute] long id)
     {
         var player = await playerService.RetrieveByIdAsync(id);
-
         if (player != null)
         {
+            logger.LogInformation("GET /players/{Id} retrieved: {@Player}", id, player);
             return TypedResults.Ok(player);
         }
         else
         {
+            logger.LogWarning("GET /players/{Id} not found", id);
             return TypedResults.NotFound();
         }
     }
@@ -120,13 +128,18 @@ public async Task<IResult> GetByIdAsync([FromRoute] long id)
     public async Task<IResult> GetBySquadNumberAsync([FromRoute] int squadNumber)
     {
         var player = await playerService.RetrieveBySquadNumberAsync(squadNumber);
-
         if (player != null)
         {
+            logger.LogInformation(
+                "GET /players/squad/{SquadNumber} retrieved: {@Player}",
+                squadNumber,
+                player
+            );
             return TypedResults.Ok(player);
         }
         else
         {
+            logger.LogWarning("GET /players/squad/{SquadNumber} not found", squadNumber);
             return TypedResults.NotFound();
         }
     }
@@ -151,7 +164,6 @@ public async Task<IResult> GetBySquadNumberAsync([FromRoute] int squadNumber)
     public async Task<IResult> PutAsync([FromRoute] long id, [FromBody] PlayerRequestModel player)
     {
         var validation = await validator.ValidateAsync(player);
-
         if (!validation.IsValid)
         {
             var errors = validation
@@ -161,14 +173,13 @@ public async Task<IResult> PutAsync([FromRoute] long id, [FromBody] PlayerReques
             logger.LogWarning("PUT /players/{Id} validation failed: {@Errors}", id, errors);
             return TypedResults.BadRequest(errors);
         }
-
         if (await playerService.RetrieveByIdAsync(id) == null)
         {
+            logger.LogWarning("PUT /players/{Id} not found", id);
             return TypedResults.NotFound();
         }
-
         await playerService.UpdateAsync(player);
-
+        logger.LogInformation("PUT /players/{Id} updated: {@Player}", id, player);
         return TypedResults.NoContent();
     }
 
@@ -189,12 +200,13 @@ public async Task<IResult> DeleteAsync([FromRoute] long id)
     {
         if (await playerService.RetrieveByIdAsync(id) == null)
         {
+            logger.LogWarning("DELETE /players/{Id} not found", id);
             return TypedResults.NotFound();
         }
         else
         {
             await playerService.DeleteAsync(id);
-
+            logger.LogInformation("DELETE /players/{Id} deleted", id);
             return TypedResults.NoContent();
         }
     }