upstream/nextcloud
1
0
Fork 0
mirror of https://github.com/nextcloud/server.git synced 2026-06-13 18:50:47 -04:00
Code Issues Projects Releases Packages Wiki Activity Actions 9

Merge pull request #60580 from nextcloud/backport/60547/stable27
Some checks failed
Cypress / init (push) Has been cancelled
Details
Lint php / php-lint (push) Has been cancelled
Details
Node tests / versions (push) Has been cancelled
Details
Node / node (push) Has been cancelled
Details
Psalm static code analysis / static-code-analysis (push) Has been cancelled
Details
Psalm static code analysis / static-code-analysis-security (push) Has been cancelled
Details
Psalm static code analysis / static-code-analysis-ocp (push) Has been cancelled
Details
Cypress / runner 1 (push) Has been cancelled
Details
Cypress / runner 2 (push) Has been cancelled
Details
Cypress / runner component (push) Has been cancelled
Details
Cypress / cypress-summary (push) Has been cancelled
Details
Lint php / php-lint-summary (push) Has been cancelled
Details
Node tests / test (push) Has been cancelled
Details
Node tests / jsunit (push) Has been cancelled
Details
Node tests / handlebars (push) Has been cancelled
Details

Browse source 
[stable27] fix: don't tell the remote their token is lower
This commit is contained in:
Stephan Orbaugh 2026-05-20 20:27:29 +02:00 committed by GitHub
commit f998ff0276
No known key found for this signature in database
GPG key ID: B5690EEEBB952194
2 changed files with 3 additions and 4 deletions
Show stats Download patch file Download diff file Expand all files Collapse all files

2
apps/federation/lib/Controller/OCSAuthAPIController.php
View file

@ -126,7 +126,7 @@ class OCSAuthAPIController extends OCSController {
'remote server (' . $url . ') presented lower token. We will initiate the exchange of the shared secret.',
['app' => 'federation']
);
throw new OCSForbiddenException();
return new DataResponse();
}
$this->jobList->add(

5
apps/federation/tests/Controller/OCSAuthAPIControllerTest.php
View file

@ -125,9 +125,9 @@ class OCSAuthAPIControllerTest extends TestCase {
try {
$this->ocsAuthApi->requestSharedSecret($url, $token);
$this->assertTrue($ok);
$this->assertTrue($isTrustedServer);
} catch (OCSForbiddenException $e) {
$this->assertFalse($ok);
$this->assertFalse($isTrustedServer);
}
}
@ -183,7 +183,6 @@ class OCSAuthAPIControllerTest extends TestCase {
try {
$result = $ocsAuthApi->getSharedSecret($url, $token);
$this->assertTrue($ok);
$data = $result->getData();
$this->assertSame('secret', $data['sharedSecret']);
} catch (OCSForbiddenException $e) {