Make password optional

back/src/Kyoo.Abstractions/Models/Resources/User.cs

@@ -54,7 +54,9 @@ namespace Kyoo.Abstractions.Models
 		/// The user password (hashed, it can't be read like that). The hashing format is implementation defined.
 		/// </summary>
 		[SerializeIgnore]
-		public string Password { get; set; }
+		public string? Password { get; set; }
+
+		public bool HasPassword => Password != null;
 
 		/// <summary>
 		/// The list of permissions of the user. The format of this is implementation dependent.

back/src/Kyoo.Authentication/Models/DTO/PasswordResetRequest.cs

@@ -28,7 +28,7 @@ public class PasswordResetRequest
 	/// <summary>
 	/// The old password
 	/// </summary>
-	public string OldPassword { get; set; }
+	public string? OldPassword { get; set; }
 
 	/// <summary>
 	/// The new password

back/src/Kyoo.Authentication/Views/AuthApi.cs

@@ -316,7 +316,7 @@ namespace Kyoo.Authentication.Views
 		public async Task<ActionResult<User>> ResetPassword([FromBody] PasswordResetRequest request)
 		{
 			User user = await users.Get(User.GetIdOrThrow());
-			if (!BCryptNet.Verify(request.OldPassword, user.Password))
+			if (user.HasPassword && !BCryptNet.Verify(request.OldPassword, user.Password))
 				return Forbid(new RequestError("The old password is invalid."));
 			return await users.Patch(
 				user.Id,

back/src/Kyoo.Postgresql/Migrations/20240302151906_MakePasswordOptional.cs

@@ -0,0 +1,38 @@
+using Microsoft.EntityFrameworkCore.Migrations;
+
+#nullable disable
+
+namespace Kyoo.Postgresql.Migrations
+{
+	/// <inheritdoc />
+	public partial class MakePasswordOptional : Migration
+	{
+		/// <inheritdoc />
+		protected override void Up(MigrationBuilder migrationBuilder)
+		{
+			migrationBuilder.AlterColumn<string>(
+				name: "password",
+				table: "users",
+				type: "text",
+				nullable: true,
+				oldClrType: typeof(string),
+				oldType: "text"
+			);
+		}
+
+		/// <inheritdoc />
+		protected override void Down(MigrationBuilder migrationBuilder)
+		{
+			migrationBuilder.AlterColumn<string>(
+				name: "password",
+				table: "users",
+				type: "text",
+				nullable: false,
+				defaultValue: "",
+				oldClrType: typeof(string),
+				oldType: "text",
+				oldNullable: true
+			);
+		}
+	}
+}

back/src/Kyoo.Postgresql/Migrations/PostgresContextModelSnapshot.cs

@@ -620,7 +620,6 @@ namespace Kyoo.Postgresql.Migrations
 						.HasColumnName("external_id");
 
 					b.Property<string>("Password")
-						.IsRequired()
 						.HasColumnType("text")
 						.HasColumnName("password");