123456789101112131415161718192021222324252627282930313233343536373839404142434445464748495051525354555657585960616263646566676869707172737475767778798081828384858687888990919293949596979899100101102103104105106107108109110111112113114115116117118119120121122123124125126127128129130131132133134135136137138139140141142143144145146147148149150151152153154155156157158159160161162163164165166167168169170171172173174175176177178179180181182183184185186187188189190191192193194195196197198199200201202203204205206207208209210211212213214215216217218219220221222223224225226227228229230231232233234235236237238239240241242243244245246247248249250251252253254255256257258259260261262263264265266267268269270271272273274275276277278279280281282283284285286287288289290291292293294295296297298299300301302303304305306307308309310311312313314 |
- <?php
- declare(strict_types=1);
- /**
- * SPDX-FileCopyrightText: 2017 Nextcloud GmbH and Nextcloud contributors
- * SPDX-License-Identifier: AGPL-3.0-or-later
- */
- 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\Attribute\AnonRateLimit;
- use OCP\AppFramework\Http\Attribute\UserRateLimit;
- use OCP\AppFramework\Http\DataResponse;
- use OCP\AppFramework\Http\TemplateResponse;
- use OCP\IRequest;
- use OCP\ISession;
- use OCP\IUser;
- use OCP\IUserSession;
- use PHPUnit\Framework\MockObject\MockObject;
- use Test\TestCase;
- class TestRateLimitController extends Controller {
- /**
- * @UserRateThrottle(limit=20, period=200)
- * @AnonRateThrottle(limit=10, period=100)
- */
- public function testMethodWithAnnotation() {
- }
- /**
- * @AnonRateThrottle(limit=10, period=100)
- */
- public function testMethodWithAnnotationFallback() {
- }
- public function testMethodWithoutAnnotation() {
- }
- #[UserRateLimit(limit: 20, period: 200)]
- #[AnonRateLimit(limit: 10, period: 100)]
- public function testMethodWithAttributes() {
- }
- #[AnonRateLimit(limit: 10, period: 100)]
- public function testMethodWithAttributesFallback() {
- }
- }
- /**
- * @group DB
- */
- class RateLimitingMiddlewareTest extends TestCase {
- private IRequest|MockObject $request;
- private IUserSession|MockObject $userSession;
- private ControllerMethodReflector $reflector;
- private Limiter|MockObject $limiter;
- private ISession|MockObject $session;
- private RateLimitingMiddleware $rateLimitingMiddleware;
- protected function setUp(): void {
- parent::setUp();
- $this->request = $this->createMock(IRequest::class);
- $this->userSession = $this->createMock(IUserSession::class);
- $this->reflector = new ControllerMethodReflector();
- $this->limiter = $this->createMock(Limiter::class);
- $this->session = $this->createMock(ISession::class);
- $this->rateLimitingMiddleware = new RateLimitingMiddleware(
- $this->request,
- $this->userSession,
- $this->reflector,
- $this->limiter,
- $this->session
- );
- }
- public function testBeforeControllerWithoutAnnotationForAnon(): void {
- $this->limiter
- ->expects($this->never())
- ->method('registerUserRequest');
- $this->limiter
- ->expects($this->never())
- ->method('registerAnonRequest');
- $this->userSession->expects($this->once())
- ->method('isLoggedIn')
- ->willReturn(false);
- /** @var TestRateLimitController|MockObject $controller */
- $controller = $this->createMock(TestRateLimitController::class);
- $this->reflector->reflect($controller, 'testMethodWithoutAnnotation');
- $this->rateLimitingMiddleware->beforeController($controller, 'testMethodWithoutAnnotation');
- }
- public function testBeforeControllerWithoutAnnotationForLoggedIn(): void {
- $this->limiter
- ->expects($this->never())
- ->method('registerUserRequest');
- $this->limiter
- ->expects($this->never())
- ->method('registerAnonRequest');
- $this->userSession->expects($this->once())
- ->method('isLoggedIn')
- ->willReturn(true);
- /** @var TestRateLimitController|MockObject $controller */
- $controller = $this->createMock(TestRateLimitController::class);
- $this->reflector->reflect($controller, 'testMethodWithoutAnnotation');
- $this->rateLimitingMiddleware->beforeController($controller, 'testMethodWithoutAnnotation');
- }
- public function testBeforeControllerForAnon(): void {
- $controller = new TestRateLimitController('test', $this->request);
- $this->request
- ->method('getRemoteAddress')
- ->willReturn('127.0.0.1');
- $this->userSession
- ->expects($this->once())
- ->method('isLoggedIn')
- ->willReturn(false);
- $this->limiter
- ->expects($this->never())
- ->method('registerUserRequest');
- $this->limiter
- ->expects($this->once())
- ->method('registerAnonRequest')
- ->with(get_class($controller) . '::testMethodWithAnnotation', '10', '100', '127.0.0.1');
- $this->reflector->reflect($controller, 'testMethodWithAnnotation');
- $this->rateLimitingMiddleware->beforeController($controller, 'testMethodWithAnnotation');
- }
- public function testBeforeControllerForLoggedIn(): void {
- $controller = new TestRateLimitController('test', $this->request);
- /** @var IUser|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->limiter
- ->expects($this->never())
- ->method('registerAnonRequest');
- $this->limiter
- ->expects($this->once())
- ->method('registerUserRequest')
- ->with(get_class($controller) . '::testMethodWithAnnotation', '20', '200', $user);
- $this->reflector->reflect($controller, 'testMethodWithAnnotation');
- $this->rateLimitingMiddleware->beforeController($controller, 'testMethodWithAnnotation');
- }
- public function testBeforeControllerAnonWithFallback(): void {
- $controller = new TestRateLimitController('test', $this->request);
- $this->request
- ->expects($this->once())
- ->method('getRemoteAddress')
- ->willReturn('127.0.0.1');
- $this->userSession
- ->expects($this->once())
- ->method('isLoggedIn')
- ->willReturn(true);
- $this->limiter
- ->expects($this->never())
- ->method('registerUserRequest');
- $this->limiter
- ->expects($this->once())
- ->method('registerAnonRequest')
- ->with(get_class($controller) . '::testMethodWithAnnotationFallback', '10', '100', '127.0.0.1');
- $this->reflector->reflect($controller, 'testMethodWithAnnotationFallback');
- $this->rateLimitingMiddleware->beforeController($controller, 'testMethodWithAnnotationFallback');
- }
- public function testBeforeControllerAttributesForAnon(): void {
- $controller = new TestRateLimitController('test', $this->request);
- $this->request
- ->method('getRemoteAddress')
- ->willReturn('127.0.0.1');
- $this->userSession
- ->expects($this->once())
- ->method('isLoggedIn')
- ->willReturn(false);
- $this->limiter
- ->expects($this->never())
- ->method('registerUserRequest');
- $this->limiter
- ->expects($this->once())
- ->method('registerAnonRequest')
- ->with(get_class($controller) . '::testMethodWithAttributes', '10', '100', '127.0.0.1');
- $this->reflector->reflect($controller, 'testMethodWithAttributes');
- $this->rateLimitingMiddleware->beforeController($controller, 'testMethodWithAttributes');
- }
- public function testBeforeControllerAttributesForLoggedIn(): void {
- $controller = new TestRateLimitController('test', $this->request);
- /** @var IUser|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->limiter
- ->expects($this->never())
- ->method('registerAnonRequest');
- $this->limiter
- ->expects($this->once())
- ->method('registerUserRequest')
- ->with(get_class($controller) . '::testMethodWithAttributes', '20', '200', $user);
- $this->reflector->reflect($controller, 'testMethodWithAttributes');
- $this->rateLimitingMiddleware->beforeController($controller, 'testMethodWithAttributes');
- }
- public function testBeforeControllerAttributesAnonWithFallback(): void {
- $controller = new TestRateLimitController('test', $this->request);
- $this->request
- ->expects($this->once())
- ->method('getRemoteAddress')
- ->willReturn('127.0.0.1');
- $this->userSession
- ->expects($this->once())
- ->method('isLoggedIn')
- ->willReturn(true);
- $this->limiter
- ->expects($this->never())
- ->method('registerUserRequest');
- $this->limiter
- ->expects($this->once())
- ->method('registerAnonRequest')
- ->with(get_class($controller) . '::testMethodWithAttributesFallback', '10', '100', '127.0.0.1');
- $this->reflector->reflect($controller, 'testMethodWithAttributesFallback');
- $this->rateLimitingMiddleware->beforeController($controller, 'testMethodWithAttributesFallback');
- }
- public function testAfterExceptionWithOtherException(): void {
- $this->expectException(\Exception::class);
- $this->expectExceptionMessage('My test exception');
- $controller = new TestRateLimitController('test', $this->request);
- $this->rateLimitingMiddleware->afterException($controller, 'testMethod', new \Exception('My test exception'));
- }
- public function testAfterExceptionWithJsonBody(): void {
- $controller = new TestRateLimitController('test', $this->request);
- $this->request
- ->expects($this->once())
- ->method('getHeader')
- ->with('Accept')
- ->willReturn('JSON');
- $result = $this->rateLimitingMiddleware->afterException($controller, 'testMethod', new RateLimitExceededException());
- $expected = new DataResponse([], 429
- );
- $this->assertEquals($expected, $result);
- }
- public function testAfterExceptionWithHtmlBody(): void {
- $controller = new TestRateLimitController('test', $this->request);
- $this->request
- ->expects($this->once())
- ->method('getHeader')
- ->with('Accept')
- ->willReturn('html');
- $result = $this->rateLimitingMiddleware->afterException($controller, 'testMethod', new RateLimitExceededException());
- $expected = new TemplateResponse(
- 'core',
- '429',
- [],
- TemplateResponse::RENDER_AS_GUEST
- );
- $expected->setStatus(429);
- $this->assertEquals($expected, $result);
- $this->assertIsString($result->render());
- }
- }
|