Maybe we should keep that backup rate-limit

This commit is contained in:
Matthew Penner 2020-12-06 13:55:45 -07:00
parent a5cebd6bcf
commit 1ce6d3bbba

View file

@ -109,13 +109,13 @@ class InitiateBackupService
*/ */
public function handle(Server $server, string $name = null, bool $override = false): Backup public function handle(Server $server, string $name = null, bool $override = false): Backup
{ {
/*$previous = $this->repository->getBackupsGeneratedDuringTimespan($server->id, 10); $previous = $this->repository->getBackupsGeneratedDuringTimespan($server->id, 10);
if ($previous->count() >= 2) { if ($previous->count() >= 2) {
throw new TooManyRequestsHttpException( throw new TooManyRequestsHttpException(
CarbonImmutable::now()->diffInSeconds($previous->last()->created_at->addMinutes(10)), CarbonImmutable::now()->diffInSeconds($previous->last()->created_at->addMinutes(10)),
'Only two backups may be generated within a 10 minute span of time.' 'Only two backups may be generated within a 10 minute span of time.'
); );
}*/ }
// Check if the server has reached or exceeded it's backup limit // Check if the server has reached or exceeded it's backup limit
if (!$server->backup_limit || $server->backups()->where('is_successful', true)->count() >= $server->backup_limit) { if (!$server->backup_limit || $server->backups()->where('is_successful', true)->count() >= $server->backup_limit) {