Merge pull request #10065 from nextcloud/rakekniven-patch-1
Update SettingsController.php
This commit is contained in:
commit
c92fe76f77
|
@ -75,7 +75,7 @@ class SettingsController extends Controller {
|
|||
string $redirectUri): JSONResponse {
|
||||
|
||||
if (filter_var($redirectUri, FILTER_VALIDATE_URL, FILTER_FLAG_SCHEME_REQUIRED|FILTER_FLAG_HOST_REQUIRED) === false) {
|
||||
return new JSONResponse(['message' => $this->l->t('Your redirect url needs to be a full url for example: https://yourdomain.com/path')], Http::STATUS_BAD_REQUEST);
|
||||
return new JSONResponse(['message' => $this->l->t('Your redirect URL needs to be a full URL for example: https://yourdomain.com/path')], Http::STATUS_BAD_REQUEST);
|
||||
}
|
||||
|
||||
$client = new Client();
|
||||
|
|
|
@ -189,6 +189,6 @@ class SettingsControllerTest extends TestCase {
|
|||
$result = $this->settingsController->addClient('test', 'invalidurl');
|
||||
|
||||
$this->assertEquals(Http::STATUS_BAD_REQUEST, $result->getStatus());
|
||||
$this->assertSame(['message' => 'Your redirect url needs to be a full url for example: https://yourdomain.com/path'], $result->getData());
|
||||
$this->assertSame(['message' => 'Your redirect URL needs to be a full URL for example: https://yourdomain.com/path'], $result->getData());
|
||||
}
|
||||
}
|
||||
|
|
Loading…
Reference in New Issue