diff --git a/3rdparty b/3rdparty index 27a56c5bb9..82d0c23acd 160000 --- a/3rdparty +++ b/3rdparty @@ -1 +1 @@ -Subproject commit 27a56c5bb9d0ec514a8fb22044fd5f03a51ea2a5 +Subproject commit 82d0c23acdce40f7b1ef3201aaff0a506b3d6129 diff --git a/apps/federation/lib/BackgroundJob/GetSharedSecret.php b/apps/federation/lib/BackgroundJob/GetSharedSecret.php index 9390b047a9..875f35899d 100644 --- a/apps/federation/lib/BackgroundJob/GetSharedSecret.php +++ b/apps/federation/lib/BackgroundJob/GetSharedSecret.php @@ -31,7 +31,6 @@ namespace OCA\Federation\BackgroundJob; use GuzzleHttp\Exception\ClientException; use GuzzleHttp\Exception\RequestException; -use GuzzleHttp\Ring\Exception\RingException; use OCA\Federation\TrustedServers; use OCP\AppFramework\Http; use OCP\AppFramework\Utility\ITimeFactory; @@ -191,14 +190,7 @@ class GetSharedSecret extends Job { 'level' => ILogger::INFO, 'app' => 'federation', ]); - } catch (RingException $e) { - $status = -1; // There is no status code if we could not connect - $this->logger->logException($e, [ - 'message' => 'Could not connect to ' . $target, - 'level' => ILogger::INFO, - 'app' => 'federation', - ]); - } catch (\Exception $e) { + } catch (\Throwable $e) { $status = Http::STATUS_INTERNAL_SERVER_ERROR; $this->logger->logException($e, ['app' => 'federation']); } diff --git a/apps/federation/lib/BackgroundJob/RequestSharedSecret.php b/apps/federation/lib/BackgroundJob/RequestSharedSecret.php index c9bee9fa6b..1f801eec56 100644 --- a/apps/federation/lib/BackgroundJob/RequestSharedSecret.php +++ b/apps/federation/lib/BackgroundJob/RequestSharedSecret.php @@ -30,7 +30,6 @@ namespace OCA\Federation\BackgroundJob; use GuzzleHttp\Exception\ClientException; use GuzzleHttp\Exception\RequestException; -use GuzzleHttp\Ring\Exception\RingException; use OCA\Federation\TrustedServers; use OCP\AppFramework\Http; use OCP\AppFramework\Utility\ITimeFactory; @@ -184,10 +183,7 @@ class RequestSharedSecret extends Job { } catch (RequestException $e) { $status = -1; // There is no status code if we could not connect $this->logger->info('Could not connect to ' . $target, ['app' => 'federation']); - } catch (RingException $e) { - $status = -1; // There is no status code if we could not connect - $this->logger->info('Could not connect to ' . $target, ['app' => 'federation']); - } catch (\Exception $e) { + } catch (\Throwable $e) { $status = Http::STATUS_INTERNAL_SERVER_ERROR; $this->logger->logException($e, ['app' => 'federation']); } diff --git a/apps/federation/tests/BackgroundJob/GetSharedSecretTest.php b/apps/federation/tests/BackgroundJob/GetSharedSecretTest.php index 7bd9208136..6da3ba6894 100644 --- a/apps/federation/tests/BackgroundJob/GetSharedSecretTest.php +++ b/apps/federation/tests/BackgroundJob/GetSharedSecretTest.php @@ -28,7 +28,6 @@ namespace OCA\Federation\Tests\BackgroundJob; use GuzzleHttp\Exception\ConnectException; -use GuzzleHttp\Ring\Exception\RingException; use OCA\Federation\BackgroundJob\GetSharedSecret; use OCA\Federation\TrustedServers; use OCA\Files_Sharing\Tests\TestCase; @@ -298,41 +297,4 @@ class GetSharedSecretTest extends TestCase { $this->assertTrue($this->invokePrivate($this->getSharedSecret, 'retainJob')); } - - public function testRunRingException() { - $target = 'targetURL'; - $source = 'sourceURL'; - $token = 'token'; - - $argument = ['url' => $target, 'token' => $token]; - - $this->timeFactory->method('getTime') - ->willReturn(42); - - $this->urlGenerator - ->expects($this->once()) - ->method('getAbsoluteURL') - ->with('/') - ->willReturn($source); - $this->httpClient->expects($this->once())->method('get') - ->with( - $target . '/ocs/v2.php/apps/federation/api/v1/shared-secret', - [ - 'query' => - [ - 'url' => $source, - 'token' => $token, - 'format' => 'json', - ], - 'timeout' => 3, - 'connect_timeout' => 3, - ] - )->willThrowException($this->createMock(RingException::class)); - - $this->trustedServers->expects($this->never())->method('addSharedSecret'); - - $this->invokePrivate($this->getSharedSecret, 'run', [$argument]); - - $this->assertTrue($this->invokePrivate($this->getSharedSecret, 'retainJob')); - } } diff --git a/apps/federation/tests/BackgroundJob/RequestSharedSecretTest.php b/apps/federation/tests/BackgroundJob/RequestSharedSecretTest.php index cb912d1676..f03c5aa2a2 100644 --- a/apps/federation/tests/BackgroundJob/RequestSharedSecretTest.php +++ b/apps/federation/tests/BackgroundJob/RequestSharedSecretTest.php @@ -27,7 +27,6 @@ namespace OCA\Federation\Tests\BackgroundJob; use GuzzleHttp\Exception\ConnectException; -use GuzzleHttp\Ring\Exception\RingException; use OCA\Federation\BackgroundJob\RequestSharedSecret; use OCA\Federation\TrustedServers; use OCP\AppFramework\Http; @@ -279,40 +278,4 @@ class RequestSharedSecretTest extends TestCase { $this->invokePrivate($this->requestSharedSecret, 'run', [$argument]); $this->assertTrue($this->invokePrivate($this->requestSharedSecret, 'retainJob')); } - - public function testRunRingException() { - $target = 'targetURL'; - $source = 'sourceURL'; - $token = 'token'; - - $argument = ['url' => $target, 'token' => $token]; - - $this->timeFactory->method('getTime')->willReturn(42); - - $this->urlGenerator - ->expects($this->once()) - ->method('getAbsoluteURL') - ->with('/') - ->willReturn($source); - - $this->httpClient - ->expects($this->once()) - ->method('post') - ->with( - $target . '/ocs/v2.php/apps/federation/api/v1/request-shared-secret', - [ - 'body' => - [ - 'url' => $source, - 'token' => $token, - 'format' => 'json', - ], - 'timeout' => 3, - 'connect_timeout' => 3, - ] - )->willThrowException($this->createMock(RingException::class)); - - $this->invokePrivate($this->requestSharedSecret, 'run', [$argument]); - $this->assertTrue($this->invokePrivate($this->requestSharedSecret, 'retainJob')); - } }