Fix: Remove files that are larger than expected (#74)

Author: RikudouSage

src/Command/DownloadCommand.php

@@ -8,6 +8,7 @@
 use App\Enum\NamingConvention;
 use App\Enum\Setting;
 use App\Exception\ExitException;
+use App\Exception\ForceRetryException;
 use App\Exception\InvalidValueException;
 use App\Exception\TooManyRetriesException;
 use App\Exception\UnreadableFileException;
@@ -33,6 +34,8 @@
 use Symfony\Component\Console\Input\InputOption;
 use Symfony\Component\Console\Output\OutputInterface;
 use Symfony\Component\Console\Style\SymfonyStyle;
+use Symfony\Component\HttpClient\Exception\ClientException;
+use Symfony\Component\HttpFoundation\Response;
 
 #[AsCommand('download')]
 final class DownloadCommand extends Command
@@ -303,10 +306,19 @@ protected function execute(InputInterface $input, OutputInterface $output): int
                             } catch (UnreadableFileException) {
                                 $hash = hash_init('md5');
                             }
-                            foreach ($responses as $response) {
-                                $chunk = $response->getContent();
-                                $writer->writeChunk($targetFile, $chunk, $chunkSize);
-                                hash_update($hash, $chunk);
+                            try {
+                                foreach ($responses as $response) {
+                                    $chunk = $response->getContent();
+                                    $writer->writeChunk($targetFile, $chunk, $chunkSize);
+                                    hash_update($hash, $chunk);
+                                }
+                            } catch (ClientException $e) {
+                                if ($e->getCode() === Response::HTTP_REQUESTED_RANGE_NOT_SATISFIABLE) {
+                                    $writer->remove($targetFile);
+                                    throw new ForceRetryException();
+                                }
+
+                                throw $e;
                             }
                             $hash = hash_final($hash);
                             $writer->finalizeWriting($targetFile, $hash);

src/Exception/ForceRetryException.php

@@ -0,0 +1,9 @@
+<?php
+
+namespace App\Exception;
+
+use RuntimeException;
+
+final class ForceRetryException extends RuntimeException
+{
+}

src/Service/RetryService.php

@@ -2,6 +2,7 @@
 
 namespace App\Service;
 
+use App\Exception\ForceRetryException;
 use App\Exception\TooManyRetriesException;
 use Exception;
 use Throwable;
@@ -27,13 +28,15 @@ public function retry(callable $callable, int $maxRetries, int $retryDelay, ?arr
 
                 return;
             } catch (Exception $e) {
-                $thrown[] = $e;
-                ++$retries;
-                if (!$this->matches($e, $exceptions)) {
-                    throw $e;
-                }
-                if ($ignoreExceptions && $this->matches($e, $ignoreExceptions)) {
-                    throw $e;
+                if (!$e instanceof ForceRetryException) {
+                    $thrown[] = $e;
+                    ++$retries;
+                    if (!$this->matches($e, $exceptions)) {
+                        throw $e;
+                    }
+                    if ($ignoreExceptions && $this->matches($e, $ignoreExceptions)) {
+                        throw $e;
+                    }
                 }
                 sleep($retryDelay);
             }