Small fixup and fix unit test conflicts

Author: FeTetra

ProjectLighthouse.Servers.GameServer/Controllers/MessageController.cs

@@ -57,7 +57,7 @@ public async Task<IActionResult> Announce()
 
         UserEntity? user = await this.database.UserFromGameToken(token);
 
-        StringBuilder announceText = new(ServerConfiguration.Instance.AnnounceText + "\n\n");
+        StringBuilder announceText = new(ServerConfiguration.Instance.AnnounceText);
 
         announceText.Replace("%user", user.Username);
         announceText.Replace("%id", token.UserId.ToString());
@@ -69,7 +69,7 @@ public async Task<IActionResult> Announce()
 
         if (EmailEnforcementConfiguration.Instance.EnableEmailEnforcement)
         {
-            announceText.Append(BaseLayoutStrings.EmailEnforcementWarnMain.Translate(LocalizationManager.DefaultLang) + "\n\n");
+            announceText.Append("\n\n" + BaseLayoutStrings.EmailEnforcementWarnMain.Translate(LocalizationManager.DefaultLang) + "\n\n");
 
             if (user.EmailAddress == null)
             {

ProjectLighthouse.Servers.GameServer/Middlewares/EmailEnforcementMiddleware.cs

@@ -15,12 +15,12 @@ public EmailEnforcementMiddleware(RequestDelegate next) : base(next)
 
     public override async Task InvokeAsync(HttpContext context, DatabaseContext database)
     {
-        // Split path into segments
-        string[] pathSegments = context.Request.Path.ToString().Split("/", StringSplitOptions.RemoveEmptyEntries);
+        if (EmailEnforcementConfiguration.Instance.EnableEmailEnforcement)
+        { 
+            // Split path into segments
+            string[] pathSegments = context.Request.Path.ToString().Split("/", StringSplitOptions.RemoveEmptyEntries);
 
-        if (pathSegments[0] == "LITTLEBIGPLANETPS3_XML")
-        {
-            if (EmailEnforcementConfiguration.Instance.EnableEmailEnforcement)
+            if (pathSegments[0] == "LITTLEBIGPLANETPS3_XML")
             {
                 // Get user via GameToken
                 GameTokenEntity? token = await database.GameTokenFromRequest(context.Request);
@@ -35,7 +35,7 @@ public override async Task InvokeAsync(HttpContext context, DatabaseContext data
                     if (user == null)
                     {
                         // Send bad request status
-                        context.Response.StatusCode = StatusCodes.Status400BadRequest;
+                        context.Response.StatusCode = StatusCodes.Status403Forbidden;
                         await context.Response.WriteAsync("Not a valid user");
 
                         // Don't go to next in pipeline

ProjectLighthouse.Tests.GameApiTests/Unit/Controllers/MessageControllerTests.cs

@@ -249,7 +249,7 @@ public async Task Filter_ShouldNotSendEmail_WhenMailEnabled_AndEmailAlreadyVerif
 
         IActionResult result = await messageController.Filter(mailMock.Object);
 
-        Assert.IsType<OkResult>(result);
+        Assert.IsType<BadRequestResult>(result);
         mailMock.Verify(x => x.SendEmailAsync(It.IsAny<string>(), It.IsAny<string>(), It.IsAny<string>()), Times.Never);
     }
 
@@ -271,7 +271,7 @@ public async Task Filter_ShouldNotSendEmail_WhenMailEnabled_AndEmailFormatInvali
 
         IActionResult result = await messageController.Filter(mailMock.Object);
 
-        Assert.IsType<OkResult>(result);
+        Assert.IsType<BadRequestResult>(result);
         mailMock.Verify(x => x.SendEmailAsync(It.IsAny<string>(), It.IsAny<string>(), It.IsAny<string>()), Times.Never);
     }
 }