Merge pull request #6 from NafisianCastle/alert-autofix-5

NafisianCastle · web-flow · commit d3347899f719 · 2025-08-22T01:33:57.000+06:00
Potential fix for code scanning alert no. 5: Exposure of private information
diff --git a/YTVidShareBackend/VideoSharingService/VideoSharingService/Controllers/UserController.cs b/YTVidShareBackend/VideoSharingService/VideoSharingService/Controllers/UserController.cs
@@ -111,7 +111,7 @@ public async Task<IActionResult> GetUser(string id)
         public async Task<IActionResult> Register([FromBody] CreateUserDTO userDTO)
         {
             var sanitizedEmail = userDTO.Email?.Replace("\r", "").Replace("\n", "");
-            _logger.LogInformation($"Registration attempt for {sanitizedEmail}");
+            _logger.LogInformation("Registration attempt received.");
             if (!ModelState.IsValid)
             {
                 _logger.LogError($"Invalid post attempt {nameof(Register)}");

Original file line number	Diff line number	Diff line change
`@@ -111,7 +111,7 @@ public async Task<IActionResult> GetUser(string id)`
`111`	`111`	`public async Task<IActionResult> Register([FromBody] CreateUserDTO userDTO)`
`112`	`112`	`{`
`113`	`113`	`var sanitizedEmail = userDTO.Email?.Replace("\r", "").Replace("\n", "");`
`114`		`- _logger.LogInformation($"Registration attempt for {sanitizedEmail}");`
	`114`	`+ _logger.LogInformation("Registration attempt received.");`
`115`	`115`	`if (!ModelState.IsValid)`
`116`	`116`	`{`
`117`	`117`	`_logger.LogError($"Invalid post attempt {nameof(Register)}");`