Fix failed transfers locking a server into a unaccessible state

Author: matthewpi

app/Http/Controllers/Api/Remote/Servers/ServerTransferController.php

@@ -12,7 +12,6 @@
 use Lcobucci\JWT\Signer\Hmac\Sha256;
 use Illuminate\Database\ConnectionInterface;
 use Pterodactyl\Http\Controllers\Controller;
-use Pterodactyl\Services\Servers\SuspensionService;
 use Pterodactyl\Repositories\Eloquent\NodeRepository;
 use Pterodactyl\Repositories\Eloquent\ServerRepository;
 use Pterodactyl\Repositories\Wings\DaemonServerRepository;
@@ -114,9 +113,8 @@ public function archive(Request $request, string $uuid)
         if (! $request->input('successful')) {
             $transfer = $server->transfer;
 
-            $transfer->forceFill([
-                'successful' => false,
-            ])->saveOrFail();
+            $transfer->successful = false;
+            $transfer->saveOrFail();
 
             $allocationIds = json_decode($transfer->new_additional_allocations);
             array_push($allocationIds, $transfer->new_allocation);
@@ -189,9 +187,19 @@ public function failure(string $uuid)
         $allocationIds = json_decode($transfer->new_additional_allocations);
         array_push($allocationIds, $transfer->new_allocation);
 
+        // Begin a transaction.
+        $this->connection->beginTransaction();
+
+        // Mark the transfer as unsuccessful.
+        $transfer->successful = false;
+        $transfer->saveOrFail();
+
         // Remove the new allocations.
         $this->allocationRepository->updateWhereIn('id', $allocationIds, ['server_id' => null]);
 
+        // Commit the transaction.
+        $this->connection->commit();
+
         return new JsonResponse([], Response::HTTP_NO_CONTENT);
     }
 
@@ -236,9 +244,6 @@ public function success(string $uuid)
             $this->writer->warning($exception);
         }
 
-        // Unsuspend the server
-        $server->load('node');
-
         return new JsonResponse([], Response::HTTP_NO_CONTENT);
     }
 }