symfony
diff --git a/‎src/Symfony/Component/Translation/Bridge/Crowdin/CrowdinProvider.php
Lines changed: 31 additions & 9 deletions b/‎src/Symfony/Component/Translation/Bridge/Crowdin/CrowdinProvider.php
Lines changed: 31 additions & 9 deletions
@@ -95,8 +95,12 @@ public function write(TranslatorBagInterface $translatorBag): void
         }
 
         foreach ($responses as $response) {
-            if (200 !== $response->getStatusCode()) {
+            if (200 !== $statusCode = $response->getStatusCode()) {
                 $this->logger->error(sprintf('Unable to upload translations to Crowdin: "%s".', $response->getContent(false)));
+
+                if (500 <= $statusCode) {
+                    throw new ProviderException('Unable to upload translations to Crowdin.', $response);
+                }
             }
         }
     }
@@ -135,9 +139,13 @@ public function read(array $domains, array $locales): TranslatorBag
                 continue;
             }
 
-            if (200 !== $response->getStatusCode()) {
+            if (200 !== $statusCode = $response->getStatusCode()) {
                 $this->logger->error(sprintf('Unable to export file: "%s".', $response->getContent(false)));
 
+                if (500 <= $statusCode) {
+                    throw new ProviderException('Unable to export file.', $response);
+                }
+
                 continue;
             }
 
@@ -146,9 +154,13 @@ public function read(array $domains, array $locales): TranslatorBag
         }
 
         foreach ($downloads as [$response, $locale, $domain]) {
-            if (200 !== $response->getStatusCode()) {
+            if (200 !== $statusCode = $response->getStatusCode()) {
                 $this->logger->error(sprintf('Unable to download file content: "%s".', $response->getContent(false)));
 
+                if (500 <= $statusCode) {
+                    throw new ProviderException('Unable to download file content.', $response);
+                }
+
                 continue;
             }
 
@@ -192,8 +204,12 @@ public function delete(TranslatorBagInterface $translatorBag): void
                 continue;
             }
 
-            if (204 !== $response->getStatusCode()) {
+            if (204 !== $statusCode = $response->getStatusCode()) {
                 $this->logger->warning(sprintf('Unable to delete string: "%s".', $response->getContent(false)));
+
+                if (500 <= $statusCode) {
+                    throw new ProviderException('Unable to delete string.', $response);
+                }
             }
         }
     }
@@ -238,9 +254,13 @@ private function addFile(string $domain, string $content): ?array
             ],
         ]);
 
-        if (201 !== $response->getStatusCode()) {
+        if (201 !== $statusCode = $response->getStatusCode()) {
             $this->logger->error(sprintf('Unable to create a File in Crowdin for domain "%s": "%s".', $domain, $response->getContent(false)));
 
+            if (500 <= $statusCode) {
+                throw new ProviderException(sprintf('Unable to create a File in Crowdin for domain "%s".', $domain), $response);
+            }
+
             return null;
         }
 
@@ -261,9 +281,13 @@ private function updateFile(int $fileId, string $domain, string $content): ?arra
             ],
         ]);
 
-        if (200 !== $response->getStatusCode()) {
+        if (200 !== $statusCode = $response->getStatusCode()) {
             $this->logger->error(sprintf('Unable to update file in Crowdin for file ID "%d" and domain "%s": "%s".', $fileId, $domain, $response->getContent(false)));
 
+            if (500 <= $statusCode) {
+                throw new ProviderException(sprintf('Unable to update file in Crowdin for file ID "%d" and domain "%s".', $fileId, $domain), $response);
+            }
+
             return null;
         }
 
@@ -324,9 +348,7 @@ private function listStrings(int $fileId, int $limit, int $offset): array
         ]);
 
         if (200 !== $response->getStatusCode()) {
-            $this->logger->error(sprintf('Unable to list strings for file %d: "%s".', $fileId, $response->getContent()));
-
-            return [];
+            throw new ProviderException(sprintf('Unable to list strings for file "%d".', $fileId), $response);
         }
 
         return $response->toArray()['data'];
Original file line number	Diff line number	Diff line change
`@@ -95,8 +95,12 @@ public function write(TranslatorBagInterface $translatorBag): void`
`95`	`95`	`}`
`96`	`96`
`97`	`97`	`foreach ($responses as $response) {`
`98`		`- if (200 !== $response->getStatusCode()) {`
	`98`	`+ if (200 !== $statusCode = $response->getStatusCode()) {`
`99`	`99`	`$this->logger->error(sprintf('Unable to upload translations to Crowdin: "%s".', $response->getContent(false)));`
	`100`	`+`
	`101`	`+ if (500 <= $statusCode) {`
	`102`	`+ throw new ProviderException('Unable to upload translations to Crowdin.', $response);`
	`103`	`+ }`
`100`	`104`	`}`
`101`	`105`	`}`
`102`	`106`	`}`
`@@ -135,9 +139,13 @@ public function read(array $domains, array $locales): TranslatorBag`
`135`	`139`	`continue;`
`136`	`140`	`}`
`137`	`141`
`138`		`- if (200 !== $response->getStatusCode()) {`
	`142`	`+ if (200 !== $statusCode = $response->getStatusCode()) {`
`139`	`143`	`$this->logger->error(sprintf('Unable to export file: "%s".', $response->getContent(false)));`
`140`	`144`
	`145`	`+ if (500 <= $statusCode) {`
	`146`	`+ throw new ProviderException('Unable to export file.', $response);`
	`147`	`+ }`
	`148`	`+`
`141`	`149`	`continue;`
`142`	`150`	`}`
`143`	`151`
`@@ -146,9 +154,13 @@ public function read(array $domains, array $locales): TranslatorBag`
`146`	`154`	`}`
`147`	`155`
`148`	`156`	`foreach ($downloads as [$response, $locale, $domain]) {`
`149`		`- if (200 !== $response->getStatusCode()) {`
	`157`	`+ if (200 !== $statusCode = $response->getStatusCode()) {`
`150`	`158`	`$this->logger->error(sprintf('Unable to download file content: "%s".', $response->getContent(false)));`
`151`	`159`
	`160`	`+ if (500 <= $statusCode) {`
	`161`	`+ throw new ProviderException('Unable to download file content.', $response);`
	`162`	`+ }`
	`163`	`+`
`152`	`164`	`continue;`
`153`	`165`	`}`
`154`	`166`
`@@ -192,8 +204,12 @@ public function delete(TranslatorBagInterface $translatorBag): void`
`192`	`204`	`continue;`
`193`	`205`	`}`
`194`	`206`
`195`		`- if (204 !== $response->getStatusCode()) {`
	`207`	`+ if (204 !== $statusCode = $response->getStatusCode()) {`
`196`	`208`	`$this->logger->warning(sprintf('Unable to delete string: "%s".', $response->getContent(false)));`
	`209`	`+`
	`210`	`+ if (500 <= $statusCode) {`
	`211`	`+ throw new ProviderException('Unable to delete string.', $response);`
	`212`	`+ }`
`197`	`213`	`}`
`198`	`214`	`}`
`199`	`215`	`}`
`@@ -238,9 +254,13 @@ private function addFile(string $domain, string $content): ?array`
`238`	`254`	`],`
`239`	`255`	`]);`
`240`	`256`
`241`		`- if (201 !== $response->getStatusCode()) {`
	`257`	`+ if (201 !== $statusCode = $response->getStatusCode()) {`
`242`	`258`	`$this->logger->error(sprintf('Unable to create a File in Crowdin for domain "%s": "%s".', $domain, $response->getContent(false)));`
`243`	`259`
	`260`	`+ if (500 <= $statusCode) {`
	`261`	`+ throw new ProviderException(sprintf('Unable to create a File in Crowdin for domain "%s".', $domain), $response);`
	`262`	`+ }`
	`263`	`+`
`244`	`264`	`return null;`
`245`	`265`	`}`
`246`	`266`
`@@ -261,9 +281,13 @@ private function updateFile(int $fileId, string $domain, string $content): ?arra`
`261`	`281`	`],`
`262`	`282`	`]);`
`263`	`283`
`264`		`- if (200 !== $response->getStatusCode()) {`
	`284`	`+ if (200 !== $statusCode = $response->getStatusCode()) {`
`265`	`285`	`$this->logger->error(sprintf('Unable to update file in Crowdin for file ID "%d" and domain "%s": "%s".', $fileId, $domain, $response->getContent(false)));`
`266`	`286`
	`287`	`+ if (500 <= $statusCode) {`
	`288`	`+ throw new ProviderException(sprintf('Unable to update file in Crowdin for file ID "%d" and domain "%s".', $fileId, $domain), $response);`
	`289`	`+ }`
	`290`	`+`
`267`	`291`	`return null;`
`268`	`292`	`}`
`269`	`293`
`@@ -324,9 +348,7 @@ private function listStrings(int $fileId, int $limit, int $offset): array`
`324`	`348`	`]);`
`325`	`349`
`326`	`350`	`if (200 !== $response->getStatusCode()) {`
`327`		`- $this->logger->error(sprintf('Unable to list strings for file %d: "%s".', $fileId, $response->getContent()));`
`328`		`-`
`329`		`- return [];`
	`351`	`+ throw new ProviderException(sprintf('Unable to list strings for file "%d".', $fileId), $response);`
`330`	`352`	`}`
`331`	`353`
`332`	`354`	`return $response->toArray()['data'];`