4ea205e262
This Pull Request introduces a SabreDAV plugin that will block all older clients than 1.6.1 to connect and sync with the ownCloud instance. This has multiple reasons: 1. Old ownCloud client versions before 1.6.0 are not properly working with sticky cookies for load balancers and thus generating sessions en masse 2. Old ownCloud client versions tend to be horrible buggy In some cases we had in 80minutes about 10'000 sessions created by a single user. While this change set does not really "fix" the problem as 3rdparty legacy clients are affected as well, it is a good work-around and hopefully should force users to update their client
117 lines
3.1 KiB
PHP
117 lines
3.1 KiB
PHP
<?php
|
|
/**
|
|
* @author Lukas Reschke <lukas@owncloud.com>
|
|
*
|
|
* @copyright Copyright (c) 2015, ownCloud, Inc.
|
|
* @license AGPL-3.0
|
|
*
|
|
* This code is free software: you can redistribute it and/or modify
|
|
* it under the terms of the GNU Affero General Public License, version 3,
|
|
* as published by the Free Software Foundation.
|
|
*
|
|
* 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, version 3,
|
|
* along with this program. If not, see <http://www.gnu.org/licenses/>
|
|
*
|
|
*/
|
|
namespace Test\Connector\Sabre;
|
|
|
|
use OC\Connector\Sabre\BlockLegacyClientPlugin;
|
|
use Test\TestCase;
|
|
use OCP\IConfig;
|
|
|
|
/**
|
|
* Class BlockLegacyClientPluginTest
|
|
*
|
|
* @package Test\Connector\Sabre
|
|
*/
|
|
class BlockLegacyClientPluginTest extends TestCase {
|
|
/** @var IConfig */
|
|
private $config;
|
|
/** @var BlockLegacyClientPlugin */
|
|
private $blockLegacyClientVersionPlugin;
|
|
|
|
public function setUp() {
|
|
parent::setUp();
|
|
|
|
$this->config = $this->getMock('\OCP\IConfig');
|
|
$this->blockLegacyClientVersionPlugin = new BlockLegacyClientPlugin($this->config);
|
|
}
|
|
|
|
/**
|
|
* @return array
|
|
*/
|
|
public function oldDesktopClientProvider() {
|
|
return [
|
|
['Mozilla/5.0 (1.5.0) mirall/1.5.0'],
|
|
['mirall/1.5.0'],
|
|
['mirall/1.5.4'],
|
|
['mirall/1.6.0'],
|
|
['Mozilla/5.0 (Bogus Text) mirall/1.6.9'],
|
|
];
|
|
}
|
|
|
|
/**
|
|
* @dataProvider oldDesktopClientProvider
|
|
* @param string $userAgent
|
|
* @expectedException \OC\ServiceUnavailableException
|
|
* @expectedExceptionMessage Unsupported client version.
|
|
*/
|
|
public function testBeforeHandlerException($userAgent) {
|
|
/** @var \Sabre\HTTP\RequestInterface $request */
|
|
$request = $this->getMock('\Sabre\HTTP\RequestInterface');
|
|
$request
|
|
->expects($this->once())
|
|
->method('getHeader')
|
|
->with('User-Agent')
|
|
->will($this->returnValue($userAgent));
|
|
|
|
$this->config
|
|
->expects($this->once())
|
|
->method('getSystemValue')
|
|
->with('minimum.supported.desktop.version', '1.7.0')
|
|
->will($this->returnValue('1.7.0'));
|
|
|
|
$this->blockLegacyClientVersionPlugin->beforeHandler($request);
|
|
}
|
|
|
|
/**
|
|
* @return array
|
|
*/
|
|
public function newAndAlternateDesktopClientProvider() {
|
|
return [
|
|
['Mozilla/5.0 (1.7.0) mirall/1.7.0'],
|
|
['mirall/1.8.3'],
|
|
['mirall/1.7.2'],
|
|
['mirall/1.7.0'],
|
|
['Mozilla/5.0 (Bogus Text) mirall/1.9.3'],
|
|
];
|
|
}
|
|
|
|
/**
|
|
* @dataProvider newAndAlternateDesktopClientProvider
|
|
* @param string $userAgent
|
|
*/
|
|
public function testBeforeHandlerSucess($userAgent) {
|
|
/** @var \Sabre\HTTP\RequestInterface $request */
|
|
$request = $this->getMock('\Sabre\HTTP\RequestInterface');
|
|
$request
|
|
->expects($this->once())
|
|
->method('getHeader')
|
|
->with('User-Agent')
|
|
->will($this->returnValue($userAgent));
|
|
|
|
$this->config
|
|
->expects($this->once())
|
|
->method('getSystemValue')
|
|
->with('minimum.supported.desktop.version', '1.7.0')
|
|
->will($this->returnValue('1.7.0'));
|
|
|
|
$this->blockLegacyClientVersionPlugin->beforeHandler($request);
|
|
}
|
|
|
|
}
|