Fix search related results in OPTIONS requests
Signed-off-by: Robin Appelman <robin@icewind.nl>
This commit is contained in:
parent
01eb420a1e
commit
821d658ec3
|
@ -123,6 +123,7 @@ return array(
|
||||||
'OCA\\DAV\\Files\\BrowserErrorPagePlugin' => $baseDir . '/../lib/Files/BrowserErrorPagePlugin.php',
|
'OCA\\DAV\\Files\\BrowserErrorPagePlugin' => $baseDir . '/../lib/Files/BrowserErrorPagePlugin.php',
|
||||||
'OCA\\DAV\\Files\\FileSearchBackend' => $baseDir . '/../lib/Files/FileSearchBackend.php',
|
'OCA\\DAV\\Files\\FileSearchBackend' => $baseDir . '/../lib/Files/FileSearchBackend.php',
|
||||||
'OCA\\DAV\\Files\\FilesHome' => $baseDir . '/../lib/Files/FilesHome.php',
|
'OCA\\DAV\\Files\\FilesHome' => $baseDir . '/../lib/Files/FilesHome.php',
|
||||||
|
'OCA\\DAV\\Files\\LazySearchBackend' => $baseDir . '/../lib/Files/LazySearchBackend.php',
|
||||||
'OCA\\DAV\\Files\\RootCollection' => $baseDir . '/../lib/Files/RootCollection.php',
|
'OCA\\DAV\\Files\\RootCollection' => $baseDir . '/../lib/Files/RootCollection.php',
|
||||||
'OCA\\DAV\\Files\\Sharing\\FilesDropPlugin' => $baseDir . '/../lib/Files/Sharing/FilesDropPlugin.php',
|
'OCA\\DAV\\Files\\Sharing\\FilesDropPlugin' => $baseDir . '/../lib/Files/Sharing/FilesDropPlugin.php',
|
||||||
'OCA\\DAV\\Files\\Sharing\\PublicLinkCheckPlugin' => $baseDir . '/../lib/Files/Sharing/PublicLinkCheckPlugin.php',
|
'OCA\\DAV\\Files\\Sharing\\PublicLinkCheckPlugin' => $baseDir . '/../lib/Files/Sharing/PublicLinkCheckPlugin.php',
|
||||||
|
|
|
@ -138,6 +138,7 @@ class ComposerStaticInitDAV
|
||||||
'OCA\\DAV\\Files\\BrowserErrorPagePlugin' => __DIR__ . '/..' . '/../lib/Files/BrowserErrorPagePlugin.php',
|
'OCA\\DAV\\Files\\BrowserErrorPagePlugin' => __DIR__ . '/..' . '/../lib/Files/BrowserErrorPagePlugin.php',
|
||||||
'OCA\\DAV\\Files\\FileSearchBackend' => __DIR__ . '/..' . '/../lib/Files/FileSearchBackend.php',
|
'OCA\\DAV\\Files\\FileSearchBackend' => __DIR__ . '/..' . '/../lib/Files/FileSearchBackend.php',
|
||||||
'OCA\\DAV\\Files\\FilesHome' => __DIR__ . '/..' . '/../lib/Files/FilesHome.php',
|
'OCA\\DAV\\Files\\FilesHome' => __DIR__ . '/..' . '/../lib/Files/FilesHome.php',
|
||||||
|
'OCA\\DAV\\Files\\LazySearchBackend' => __DIR__ . '/..' . '/../lib/Files/LazySearchBackend.php',
|
||||||
'OCA\\DAV\\Files\\RootCollection' => __DIR__ . '/..' . '/../lib/Files/RootCollection.php',
|
'OCA\\DAV\\Files\\RootCollection' => __DIR__ . '/..' . '/../lib/Files/RootCollection.php',
|
||||||
'OCA\\DAV\\Files\\Sharing\\FilesDropPlugin' => __DIR__ . '/..' . '/../lib/Files/Sharing/FilesDropPlugin.php',
|
'OCA\\DAV\\Files\\Sharing\\FilesDropPlugin' => __DIR__ . '/..' . '/../lib/Files/Sharing/FilesDropPlugin.php',
|
||||||
'OCA\\DAV\\Files\\Sharing\\PublicLinkCheckPlugin' => __DIR__ . '/..' . '/../lib/Files/Sharing/PublicLinkCheckPlugin.php',
|
'OCA\\DAV\\Files\\Sharing\\PublicLinkCheckPlugin' => __DIR__ . '/..' . '/../lib/Files/Sharing/PublicLinkCheckPlugin.php',
|
||||||
|
|
|
@ -56,6 +56,9 @@ class AnonymousOptionsPlugin extends ServerPlugin {
|
||||||
// setup a fake tree for anonymous access
|
// setup a fake tree for anonymous access
|
||||||
$this->server->tree = new Tree(new Directory(''));
|
$this->server->tree = new Tree(new Directory(''));
|
||||||
$corePlugin->httpOptions($request, $response);
|
$corePlugin->httpOptions($request, $response);
|
||||||
|
$this->server->emit('afterMethod', [$request, $response]);
|
||||||
|
$this->server->emit('afterMethod:OPTIONS', [$request, $response]);
|
||||||
|
|
||||||
$this->server->sapi->sendResponse($response);
|
$this->server->sapi->sendResponse($response);
|
||||||
return false;
|
return false;
|
||||||
}
|
}
|
||||||
|
|
|
@ -0,0 +1,72 @@
|
||||||
|
<?php
|
||||||
|
/**
|
||||||
|
* @copyright Copyright (c) 2018 Robin Appelman <robin@icewind.nl>
|
||||||
|
*
|
||||||
|
* @author Robin Appelman <robin@icewind.nl>
|
||||||
|
*
|
||||||
|
* @license GNU AGPL version 3 or any later version
|
||||||
|
*
|
||||||
|
* This program is free software: you can redistribute it and/or modify
|
||||||
|
* it under the terms of the GNU Affero General Public License as
|
||||||
|
* published by the Free Software Foundation, either version 3 of the
|
||||||
|
* License, or (at your option) any later version.
|
||||||
|
*
|
||||||
|
* This program is distributed in the hope that it will be useful,
|
||||||
|
* but WITHOUT ANY WARRANTY; without even the implied warranty of
|
||||||
|
* MERCHANTABILITY or FITNESS FOR A PARTICULAR PURPOSE. See the
|
||||||
|
* GNU Affero General Public License for more details.
|
||||||
|
*
|
||||||
|
* You should have received a copy of the GNU Affero General Public License
|
||||||
|
* along with this program. If not, see <http://www.gnu.org/licenses/>.
|
||||||
|
*
|
||||||
|
*/
|
||||||
|
|
||||||
|
namespace OCA\DAV\Files;
|
||||||
|
|
||||||
|
use SearchDAV\Backend\ISearchBackend;
|
||||||
|
use SearchDAV\Query\Query;
|
||||||
|
|
||||||
|
class LazySearchBackend implements ISearchBackend {
|
||||||
|
/**
|
||||||
|
* @var ISearchBackend $backend
|
||||||
|
*/
|
||||||
|
private $backend = null;
|
||||||
|
|
||||||
|
public function setBackend(ISearchBackend $backend) {
|
||||||
|
$this->backend = $backend;
|
||||||
|
}
|
||||||
|
|
||||||
|
public function getArbiterPath() {
|
||||||
|
if ($this->backend) {
|
||||||
|
return $this->backend->getArbiterPath();
|
||||||
|
} else {
|
||||||
|
return '';
|
||||||
|
}
|
||||||
|
}
|
||||||
|
|
||||||
|
public function isValidScope($href, $depth, $path) {
|
||||||
|
if ($this->backend) {
|
||||||
|
return $this->backend->getArbiterPath();
|
||||||
|
} else {
|
||||||
|
return false;
|
||||||
|
}
|
||||||
|
}
|
||||||
|
|
||||||
|
public function getPropertyDefinitionsForScope($href, $path) {
|
||||||
|
if ($this->backend) {
|
||||||
|
return $this->backend->getArbiterPath();
|
||||||
|
} else {
|
||||||
|
return [];
|
||||||
|
}
|
||||||
|
}
|
||||||
|
|
||||||
|
public function search(Query $query) {
|
||||||
|
if ($this->backend) {
|
||||||
|
return $this->backend->getArbiterPath();
|
||||||
|
} else {
|
||||||
|
return [];
|
||||||
|
}
|
||||||
|
}
|
||||||
|
|
||||||
|
|
||||||
|
}
|
|
@ -55,6 +55,7 @@ use OCA\DAV\DAV\CustomPropertiesBackend;
|
||||||
use OCA\DAV\Connector\Sabre\QuotaPlugin;
|
use OCA\DAV\Connector\Sabre\QuotaPlugin;
|
||||||
use OCA\DAV\Files\BrowserErrorPagePlugin;
|
use OCA\DAV\Files\BrowserErrorPagePlugin;
|
||||||
use OCA\DAV\Connector\Sabre\AnonymousOptionsPlugin;
|
use OCA\DAV\Connector\Sabre\AnonymousOptionsPlugin;
|
||||||
|
use OCA\DAV\Files\LazySearchBackend;
|
||||||
use OCA\DAV\SystemTag\SystemTagPlugin;
|
use OCA\DAV\SystemTag\SystemTagPlugin;
|
||||||
use OCA\DAV\Upload\ChunkingPlugin;
|
use OCA\DAV\Upload\ChunkingPlugin;
|
||||||
use OCP\IRequest;
|
use OCP\IRequest;
|
||||||
|
@ -197,8 +198,11 @@ class Server {
|
||||||
$this->server->addPlugin(new BrowserErrorPagePlugin());
|
$this->server->addPlugin(new BrowserErrorPagePlugin());
|
||||||
}
|
}
|
||||||
|
|
||||||
|
$lazySearchBackend = new LazySearchBackend();
|
||||||
|
$this->server->addPlugin(new SearchPlugin($lazySearchBackend));
|
||||||
|
|
||||||
// wait with registering these until auth is handled and the filesystem is setup
|
// wait with registering these until auth is handled and the filesystem is setup
|
||||||
$this->server->on('beforeMethod', function () use ($root) {
|
$this->server->on('beforeMethod', function () use ($root, $lazySearchBackend) {
|
||||||
// custom properties plugin must be the last one
|
// custom properties plugin must be the last one
|
||||||
$userSession = \OC::$server->getUserSession();
|
$userSession = \OC::$server->getUserSession();
|
||||||
$user = $userSession->getUser();
|
$user = $userSession->getUser();
|
||||||
|
@ -257,13 +261,13 @@ class Server {
|
||||||
\OC::$server->getGroupManager(),
|
\OC::$server->getGroupManager(),
|
||||||
$userFolder
|
$userFolder
|
||||||
));
|
));
|
||||||
$this->server->addPlugin(new SearchPlugin(new \OCA\DAV\Files\FileSearchBackend(
|
$lazySearchBackend->setBackend(new \OCA\DAV\Files\FileSearchBackend(
|
||||||
$this->server->tree,
|
$this->server->tree,
|
||||||
$user,
|
$user,
|
||||||
\OC::$server->getRootFolder(),
|
\OC::$server->getRootFolder(),
|
||||||
\OC::$server->getShareManager(),
|
\OC::$server->getShareManager(),
|
||||||
$view
|
$view
|
||||||
)));
|
));
|
||||||
}
|
}
|
||||||
$this->server->addPlugin(new \OCA\DAV\CalDAV\BirthdayCalendar\EnablePlugin(
|
$this->server->addPlugin(new \OCA\DAV\CalDAV\BirthdayCalendar\EnablePlugin(
|
||||||
\OC::$server->getConfig(),
|
\OC::$server->getConfig(),
|
||||||
|
|
Loading…
Reference in New Issue