Merge branch 'main' of https://github.com/GeneralsOnlineDevelopmentTeam/Services

Author: x64-dev

GenOnlineService/Database/Database.Social.cs

@@ -198,6 +198,14 @@ public static async Task CreateFriendship(AppDbContext db, long userId1, long us
 		{
 			try
 			{
+				// Check if friendship already exists (handles duplicate calls / race conditions)
+				bool alreadyExists = await db.Friends.AnyAsync(f =>
+					(f.UserId1 == userId1 && f.UserId2 == userId2) ||
+					(f.UserId1 == userId2 && f.UserId2 == userId1));
+
+				if (alreadyExists)
+					return;
+
 				db.Friends.Add(new FriendEntry
 				{
 					UserId1 = userId1,
@@ -208,6 +216,12 @@ public static async Task CreateFriendship(AppDbContext db, long userId1, long us
 			}
 			catch (Exception ex)
 			{
+				// If two concurrent calls both passed the existence check, MySQL will throw a
+				// duplicate-key error (ER_DUP_ENTRY, code 1062). The friendship was already
+				// created by the other call, so this is not an error worth reporting.
+				if (ex.InnerException is MySqlConnector.MySqlException mysqlEx && mysqlEx.Number == 1062)
+					return;
+
 				Console.WriteLine($"[ERROR] CreateFriendship failed: {ex.Message}");
 				SentrySdk.CaptureException(ex);
 			}