Merge pull request 'dev' (#4) from dev into prod

Reviewed-on: #4

PortBlog.API/Controllers/AuthController.cs

@@ -59,7 +59,7 @@ namespace PortBlog.API.Controllers
             catch (Exception ex)
             {
                 logger.LogCritical(ex, "Exception while sending OTP for {ToEmail}.", messageSendDto.ToEmail);
-                return StatusCode(500, ex.Message);
+                return StatusCode(500, "A problem happened while handling your request.");
             }
         }
 

PortBlog.API/Controllers/CvController.cs

@@ -262,8 +262,8 @@ namespace PortBlog.API.Controllers
             }
             catch (Exception ex)
             {
-                _logger.LogCritical(ex, "Exception while sending message from {Name}.", message.Name);
+                _logger.LogCritical($"Exception while sending message from {message.Name}.", ex);
-                return StatusCode(500, ex.Message);
+                return StatusCode(500, "A problem happened while handling your request.");
             }
         }
     }

PortBlog.API/Program.cs

@@ -33,25 +33,13 @@ Log.Logger = loggerConfiguration
     .WriteTo.File("logs/portblog.txt", rollingInterval: RollingInterval.Day)
     .CreateLogger();
 
-var urlsSection = builder.Configuration.GetSection("Urls");
+var urls = builder.Configuration.GetSection("Urls");
-var urlsValue = urlsSection.Value;
 
-if (!string.IsNullOrWhiteSpace(urlsValue))
+if (!string.IsNullOrEmpty(urls.Value))
 {
-    var allowedUrlsToUse = urlsValue
+    var allowedUrlsToUse = urls.Value.Split(',');
-        .Split(',', StringSplitOptions.RemoveEmptyEntries | StringSplitOptions.TrimEntries)
-        .Where(u => Uri.IsWellFormedUriString(u, UriKind.Absolute))
-        .ToArray();
 
-    if (allowedUrlsToUse.Length > 0)
-    {
     builder.WebHost.UseUrls(allowedUrlsToUse);
-    }
-    else
-    {
-        // optionally log or throw depending on desired behavior
-        Log.Warning("Configured 'Urls' section was present but no valid URLs were found.");
-    }
 }
 
 var allowedCorsOrigins = builder.Configuration.GetSection("AllowedCorsOrigins");

PortBlog.API/Services/MailService.cs

@@ -67,14 +67,16 @@ namespace PortBlog.API.Services
             }
             catch (Exception ex)
             {
-                // Log full exception and persist failed message.
+                logger.LogCritical(
-                logger.LogCritical(ex, "Exception while sending mail from {FromEmail} ({Name}).", messageSendDto.FromEmail, messageSendDto.Name);
+                    "Exception while sending mail from {FromEmail} ({Name}): {Exception}",
+                    messageSendDto.FromEmail,
+                    messageSendDto.Name,
+                    ex.Message
+                );
                 messageSendDto.SentStatus = (int)MailConstants.MailStatus.Failed;
                 mailRepository.AddMessage(messageEntity);
                 await mailRepository.SaveChangesAsync();
-
+                throw new Exception();
-                // Throw a descriptive exception so callers can return an error message to the client.
-                throw new InvalidOperationException($"Failed to send email to '{messageSendDto.ToEmail}'. See inner exception for details.", ex);
             }
         }
     }