github/pyrodactyl
1
1
Fork 0
mirror of https://github.com/pyrohost/pyrodactyl.git synced 2026-04-06 04:01:58 +02:00
Code Issues 44 Packages Projects Releases Wiki Activity

feat: Added Pre-Check to the Scheduled Backup section

Browse Source 
A pre-check stage has been added to prevent the user from exceeding the limit, and this stage temporarily prevents the user from exceeding their limits.
This commit is contained in:
Luii
2026-01-20 16:28:06 +03:00
parent 960b8577a9
commit 5ab808c5a8
1 changed files with 17 additions and 0 deletions
Download Patch File Download Diff File Expand all files Collapse all files

17
app/Jobs/Schedule/RunTaskJob.php
View File

@@ -3,6 +3,7 @@
namespace Pterodactyl\Jobs\Schedule;
use Exception;
use Pterodactyl\Exceptions\Service\Backup\BackupFailedException;
use Pterodactyl\Jobs\Job;
use Carbon\CarbonImmutable;
use Pterodactyl\Models\Task;
@@ -66,6 +67,22 @@ class RunTaskJob extends Job implements ShouldQueue
$commandRepository->setServer($server)->send($this->task->payload);
break;
case Task::ACTION_BACKUP:
// A pre-check stage has been added because transactions are sent directly to Elytra without verification.
// This can be considered a temporary solution for now. In future processes, offering Elytra with better integration will play a role in removing this pre-check stage, and such transactions will be better controlled before they go to the server.
if ($server->hasBackupCountLimit() && $server->backup_limit < ($server->backups_count + 1)) {
Log::warning('Scheduled backup blocked due to backup limit', [
'task_id' => $this->task->id,
'schedule_id' => $this->task->schedule_id,
'server_id' => $server->id
]);
// TooManyBackupsException is currently scoped to daemon-level backup services,
// therefore BackupFailedException is used here to properly fail the scheduled task.
throw new BackupFailedException(
'The permitted backup limit has been exceeded.'
);
}
$affectedRows = Task::where('id', $this->task->id)
->where('is_processing', false)
->update(['is_processing' => true]);