src/Controllers/Common/AuthenticationController.cs

@@ -48,13 +48,8 @@ public class AuthenticationController : Controller {
         } else {
             user = ctx.Users.FirstOrDefault(e => e.Username == data.UserName);
         }
-
-        if (user is null) {
-            return Ok(new ParentLoginInfo { Status = MembershipUserStatus.InvalidUserName });
-        }
-
         PasswordVerificationResult result = new PasswordHasher<object>().VerifyHashedPassword(null, user.Password, data.Password);
-        if (result == PasswordVerificationResult.Failed) {
+        if (user is null || result == PasswordVerificationResult.Failed) {
             return Ok(new ParentLoginInfo { Status = MembershipUserStatus.InvalidPassword });
         }
 

src/Dockerfile

@@ -1,5 +1,5 @@
 # Use the official .NET SDK image for building the application
-FROM mcr.microsoft.com/dotnet/sdk:9.0 AS build
+FROM mcr.microsoft.com/dotnet/sdk:6.0 AS build
 WORKDIR /src
 
 # Copy the source code
@@ -13,7 +13,7 @@ RUN mv src/mods /app && ln -s /app/mods src/
 RUN mv src/assets /app && ln -s /app/assets src/
 
 # Create clean run environment (without source and sdk)
-# FROM mcr.microsoft.com/dotnet/aspnet:9.0 AS base
+# FROM mcr.microsoft.com/dotnet/aspnet:6.0 AS base
 # WORKDIR /app
 # COPY --from=build /app .
 

src/Schema/Mission.cs

@@ -3,7 +3,7 @@
 namespace sodoff.Schema;
 
 [XmlRoot(ElementName = "Mission", Namespace = "")]
-[Serializable]
+[Serializable] // FIXME: Remove serializable once we have a different way of deep copying than BinaryFormatter
 public class Mission {
     public Mission() {}