123456789101112131415161718192021222324252627282930313233343536373839404142434445464748495051525354555657585960616263646566676869707172737475767778798081828384858687888990919293949596979899100101102103104105106107108109110111112113114115116117118119120121122123124125126127128129130131132133134135136137138139140141142143144145146147148149150151152153154155156157158159160161162163164165166167168169170171172173174175176177178179180181182183184185186187188189190191192193194195196197198199200201202203204205206207208209210211212213214215216217218219220221222223224225226227228229230231232233234235236237238239240241242243244245246247248249250251252253254255256257258259260261262263264265266267268269270271272273274275276277278279280281282283 |
- <?php
- /**
- * @copyright Copyright (c) 2017 Lukas Reschke <lukas@statuscode.ch>
- *
- * @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 Test\AppFramework\Middleware\Security;
- use OC\AppFramework\Middleware\Security\RateLimitingMiddleware;
- use OC\AppFramework\Utility\ControllerMethodReflector;
- use OC\Security\RateLimiting\Exception\RateLimitExceededException;
- use OC\Security\RateLimiting\Limiter;
- use OCP\AppFramework\Controller;
- use OCP\AppFramework\Http\JSONResponse;
- use OCP\AppFramework\Http\TemplateResponse;
- use OCP\IRequest;
- use OCP\IUser;
- use OCP\IUserSession;
- use Test\TestCase;
- class RateLimitingMiddlewareTest extends TestCase {
- /** @var IRequest|\PHPUnit_Framework_MockObject_MockObject */
- private $request;
- /** @var IUserSession|\PHPUnit_Framework_MockObject_MockObject */
- private $userSession;
- /** @var ControllerMethodReflector|\PHPUnit_Framework_MockObject_MockObject */
- private $reflector;
- /** @var Limiter|\PHPUnit_Framework_MockObject_MockObject */
- private $limiter;
- /** @var RateLimitingMiddleware */
- private $rateLimitingMiddleware;
- public function setUp() {
- parent::setUp();
- $this->request = $this->createMock(IRequest::class);
- $this->userSession = $this->createMock(IUserSession::class);
- $this->reflector = $this->createMock(ControllerMethodReflector::class);
- $this->limiter = $this->createMock(Limiter::class);
- $this->rateLimitingMiddleware = new RateLimitingMiddleware(
- $this->request,
- $this->userSession,
- $this->reflector,
- $this->limiter
- );
- }
- public function testBeforeControllerWithoutAnnotation() {
- $this->reflector
- ->expects($this->at(0))
- ->method('getAnnotationParameter')
- ->with('AnonRateThrottle', 'limit')
- ->willReturn('');
- $this->reflector
- ->expects($this->at(1))
- ->method('getAnnotationParameter')
- ->with('AnonRateThrottle', 'period')
- ->willReturn('');
- $this->reflector
- ->expects($this->at(2))
- ->method('getAnnotationParameter')
- ->with('UserRateThrottle', 'limit')
- ->willReturn('');
- $this->reflector
- ->expects($this->at(3))
- ->method('getAnnotationParameter')
- ->with('UserRateThrottle', 'period')
- ->willReturn('');
- $this->limiter
- ->expects($this->never())
- ->method('registerUserRequest');
- $this->limiter
- ->expects($this->never())
- ->method('registerAnonRequest');
- /** @var Controller|\PHPUnit_Framework_MockObject_MockObject $controller */
- $controller = $this->createMock(Controller::class);
- $this->rateLimitingMiddleware->beforeController($controller, 'testMethod');
- }
- public function testBeforeControllerForAnon() {
- /** @var Controller|\PHPUnit_Framework_MockObject_MockObject $controller */
- $controller = $this->createMock(Controller::class);
- $this->request
- ->expects($this->once())
- ->method('getRemoteAddress')
- ->willReturn('127.0.0.1');
- $this->reflector
- ->expects($this->at(0))
- ->method('getAnnotationParameter')
- ->with('AnonRateThrottle', 'limit')
- ->willReturn('100');
- $this->reflector
- ->expects($this->at(1))
- ->method('getAnnotationParameter')
- ->with('AnonRateThrottle', 'period')
- ->willReturn('10');
- $this->reflector
- ->expects($this->at(2))
- ->method('getAnnotationParameter')
- ->with('UserRateThrottle', 'limit')
- ->willReturn('');
- $this->reflector
- ->expects($this->at(3))
- ->method('getAnnotationParameter')
- ->with('UserRateThrottle', 'period')
- ->willReturn('');
- $this->limiter
- ->expects($this->never())
- ->method('registerUserRequest');
- $this->limiter
- ->expects($this->once())
- ->method('registerAnonRequest')
- ->with(get_class($controller) . '::testMethod', '100', '10', '127.0.0.1');
- $this->rateLimitingMiddleware->beforeController($controller, 'testMethod');
- }
- public function testBeforeControllerForLoggedIn() {
- /** @var Controller|\PHPUnit_Framework_MockObject_MockObject $controller */
- $controller = $this->createMock(Controller::class);
- /** @var IUser|\PHPUnit_Framework_MockObject_MockObject $user */
- $user = $this->createMock(IUser::class);
- $this->userSession
- ->expects($this->once())
- ->method('isLoggedIn')
- ->willReturn(true);
- $this->userSession
- ->expects($this->once())
- ->method('getUser')
- ->willReturn($user);
- $this->reflector
- ->expects($this->at(0))
- ->method('getAnnotationParameter')
- ->with('AnonRateThrottle', 'limit')
- ->willReturn('');
- $this->reflector
- ->expects($this->at(1))
- ->method('getAnnotationParameter')
- ->with('AnonRateThrottle', 'period')
- ->willReturn('');
- $this->reflector
- ->expects($this->at(2))
- ->method('getAnnotationParameter')
- ->with('UserRateThrottle', 'limit')
- ->willReturn('100');
- $this->reflector
- ->expects($this->at(3))
- ->method('getAnnotationParameter')
- ->with('UserRateThrottle', 'period')
- ->willReturn('10');
- $this->limiter
- ->expects($this->never())
- ->method('registerAnonRequest');
- $this->limiter
- ->expects($this->once())
- ->method('registerUserRequest')
- ->with(get_class($controller) . '::testMethod', '100', '10', $user);
- $this->rateLimitingMiddleware->beforeController($controller, 'testMethod');
- }
- public function testBeforeControllerAnonWithFallback() {
- /** @var Controller|\PHPUnit_Framework_MockObject_MockObject $controller */
- $controller = $this->createMock(Controller::class);
- $this->request
- ->expects($this->once())
- ->method('getRemoteAddress')
- ->willReturn('127.0.0.1');
- $this->userSession
- ->expects($this->once())
- ->method('isLoggedIn')
- ->willReturn(false);
- $this->reflector
- ->expects($this->at(0))
- ->method('getAnnotationParameter')
- ->with('AnonRateThrottle', 'limit')
- ->willReturn('200');
- $this->reflector
- ->expects($this->at(1))
- ->method('getAnnotationParameter')
- ->with('AnonRateThrottle', 'period')
- ->willReturn('20');
- $this->reflector
- ->expects($this->at(2))
- ->method('getAnnotationParameter')
- ->with('UserRateThrottle', 'limit')
- ->willReturn('100');
- $this->reflector
- ->expects($this->at(3))
- ->method('getAnnotationParameter')
- ->with('UserRateThrottle', 'period')
- ->willReturn('10');
- $this->limiter
- ->expects($this->never())
- ->method('registerUserRequest');
- $this->limiter
- ->expects($this->once())
- ->method('registerAnonRequest')
- ->with(get_class($controller) . '::testMethod', '200', '20', '127.0.0.1');
- $this->rateLimitingMiddleware->beforeController($controller, 'testMethod');
- }
- /**
- * @expectedException \Exception
- * @expectedExceptionMessage My test exception
- */
- public function testAfterExceptionWithOtherException() {
- /** @var Controller|\PHPUnit_Framework_MockObject_MockObject $controller */
- $controller = $this->createMock(Controller::class);
- $this->rateLimitingMiddleware->afterException($controller, 'testMethod', new \Exception('My test exception'));
- }
- public function testAfterExceptionWithJsonBody() {
- /** @var Controller|\PHPUnit_Framework_MockObject_MockObject $controller */
- $controller = $this->createMock(Controller::class);
- $this->request
- ->expects($this->once())
- ->method('getHeader')
- ->with('Accept')
- ->willReturn('JSON');
- $result = $this->rateLimitingMiddleware->afterException($controller, 'testMethod', new RateLimitExceededException());
- $expected = new JSONResponse(
- [
- 'message' => 'Rate limit exceeded',
- ],
- 429
- );
- $this->assertEquals($expected, $result);
- }
- public function testAfterExceptionWithHtmlBody() {
- /** @var Controller|\PHPUnit_Framework_MockObject_MockObject $controller */
- $controller = $this->createMock(Controller::class);
- $this->request
- ->expects($this->once())
- ->method('getHeader')
- ->with('Accept')
- ->willReturn('html');
- $result = $this->rateLimitingMiddleware->afterException($controller, 'testMethod', new RateLimitExceededException());
- $expected = new TemplateResponse(
- 'core',
- '403',
- [
- 'file' => 'Rate limit exceeded',
- ],
- 'guest'
- );
- $expected->setStatus(429);
- $this->assertEquals($expected, $result);
- }
- }
|