123456789101112131415161718192021222324252627282930313233343536373839404142434445464748495051525354555657585960616263646566676869707172737475767778798081828384858687888990919293949596979899100101102103104105106107108109110111112113114115116117118119120121122123124125126127128129130131132133134135136137138139140141142143144145146147148149150151152153154155156157158159160161162163164165166167168169170171172173174175176177178179180181182183184185186187188189190191192193194195196197198199200201202203204205206207208209210211212213214215216217218219220221222223224225226227228229230231232233234235236237238239240241242243244245246247248249250251252253254255256257258259260261262263264265266267268269270271272273274275276277278279280281282283284285286287288289290291292293294295296297298299300301302303304305306307308309310 |
- <?php
- /**
- * 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\BruteForceMiddleware;
- use OC\AppFramework\Utility\ControllerMethodReflector;
- use OCP\AppFramework\Controller;
- use OCP\AppFramework\Http\Attribute\BruteForceProtection;
- use OCP\AppFramework\Http\Response;
- use OCP\IRequest;
- use OCP\Security\Bruteforce\IThrottler;
- use Psr\Log\LoggerInterface;
- use Test\TestCase;
- class TestController extends Controller {
- /**
- * @BruteForceProtection(action=login)
- */
- public function testMethodWithAnnotation() {
- }
- public function testMethodWithoutAnnotation() {
- }
- #[BruteForceProtection(action: 'single')]
- public function singleAttribute(): void {
- }
- #[BruteForceProtection(action: 'first')]
- #[BruteForceProtection(action: 'second')]
- public function multipleAttributes(): void {
- }
- }
- class BruteForceMiddlewareTest extends TestCase {
- /** @var ControllerMethodReflector */
- private $reflector;
- /** @var IThrottler|\PHPUnit\Framework\MockObject\MockObject */
- private $throttler;
- /** @var IRequest|\PHPUnit\Framework\MockObject\MockObject */
- private $request;
- /** @var LoggerInterface|\PHPUnit\Framework\MockObject\MockObject */
- private $logger;
- private BruteForceMiddleware $bruteForceMiddleware;
- protected function setUp(): void {
- parent::setUp();
- $this->reflector = new ControllerMethodReflector();
- $this->throttler = $this->createMock(IThrottler::class);
- $this->request = $this->createMock(IRequest::class);
- $this->logger = $this->createMock(LoggerInterface::class);
- $this->bruteForceMiddleware = new BruteForceMiddleware(
- $this->reflector,
- $this->throttler,
- $this->request,
- $this->logger,
- );
- }
- public function testBeforeControllerWithAnnotation(): void {
- $this->request
- ->expects($this->once())
- ->method('getRemoteAddress')
- ->willReturn('127.0.0.1');
- $this->throttler
- ->expects($this->once())
- ->method('sleepDelayOrThrowOnMax')
- ->with('127.0.0.1', 'login');
- $controller = new TestController('test', $this->request);
- $this->reflector->reflect($controller, 'testMethodWithAnnotation');
- $this->bruteForceMiddleware->beforeController($controller, 'testMethodWithAnnotation');
- }
- public function testBeforeControllerWithSingleAttribute(): void {
- $this->request
- ->expects($this->once())
- ->method('getRemoteAddress')
- ->willReturn('::1');
- $this->throttler
- ->expects($this->once())
- ->method('sleepDelayOrThrowOnMax')
- ->with('::1', 'single');
- $controller = new TestController('test', $this->request);
- $this->reflector->reflect($controller, 'singleAttribute');
- $this->bruteForceMiddleware->beforeController($controller, 'singleAttribute');
- }
- public function testBeforeControllerWithMultipleAttributes(): void {
- $this->request
- ->expects($this->once())
- ->method('getRemoteAddress')
- ->willReturn('::1');
- $this->throttler
- ->expects($this->exactly(2))
- ->method('sleepDelayOrThrowOnMax')
- ->withConsecutive(
- ['::1', 'first'],
- ['::1', 'second'],
- );
- $controller = new TestController('test', $this->request);
- $this->reflector->reflect($controller, 'multipleAttributes');
- $this->bruteForceMiddleware->beforeController($controller, 'multipleAttributes');
- }
- public function testBeforeControllerWithoutAnnotation(): void {
- $this->request
- ->expects($this->never())
- ->method('getRemoteAddress');
- $this->throttler
- ->expects($this->never())
- ->method('sleepDelayOrThrowOnMax');
- $controller = new TestController('test', $this->request);
- $this->reflector->reflect($controller, 'testMethodWithoutAnnotation');
- $this->bruteForceMiddleware->beforeController($controller, 'testMethodWithoutAnnotation');
- }
- public function testAfterControllerWithAnnotationAndThrottledRequest(): void {
- /** @var Response|\PHPUnit\Framework\MockObject\MockObject $response */
- $response = $this->createMock(Response::class);
- $response
- ->expects($this->once())
- ->method('isThrottled')
- ->willReturn(true);
- $response
- ->expects($this->once())
- ->method('getThrottleMetadata')
- ->willReturn([]);
- $this->request
- ->expects($this->once())
- ->method('getRemoteAddress')
- ->willReturn('127.0.0.1');
- $this->throttler
- ->expects($this->once())
- ->method('sleepDelayOrThrowOnMax')
- ->with('127.0.0.1', 'login');
- $this->throttler
- ->expects($this->once())
- ->method('registerAttempt')
- ->with('login', '127.0.0.1');
- $controller = new TestController('test', $this->request);
- $this->reflector->reflect($controller, 'testMethodWithAnnotation');
- $this->bruteForceMiddleware->afterController($controller, 'testMethodWithAnnotation', $response);
- }
- public function testAfterControllerWithAnnotationAndNotThrottledRequest(): void {
- /** @var Response|\PHPUnit\Framework\MockObject\MockObject $response */
- $response = $this->createMock(Response::class);
- $response
- ->expects($this->once())
- ->method('isThrottled')
- ->willReturn(false);
- $this->request
- ->expects($this->never())
- ->method('getRemoteAddress');
- $this->throttler
- ->expects($this->never())
- ->method('sleepDelayOrThrowOnMax');
- $this->throttler
- ->expects($this->never())
- ->method('registerAttempt');
- $controller = new TestController('test', $this->request);
- $this->reflector->reflect($controller, 'testMethodWithAnnotation');
- $this->bruteForceMiddleware->afterController($controller, 'testMethodWithAnnotation', $response);
- }
- public function testAfterControllerWithSingleAttribute(): void {
- /** @var Response|\PHPUnit\Framework\MockObject\MockObject $response */
- $response = $this->createMock(Response::class);
- $response
- ->expects($this->once())
- ->method('isThrottled')
- ->willReturn(true);
- $response
- ->expects($this->once())
- ->method('getThrottleMetadata')
- ->willReturn([]);
- $this->request
- ->expects($this->once())
- ->method('getRemoteAddress')
- ->willReturn('::1');
- $this->throttler
- ->expects($this->once())
- ->method('sleepDelayOrThrowOnMax')
- ->with('::1', 'single');
- $this->throttler
- ->expects($this->once())
- ->method('registerAttempt')
- ->with('single', '::1');
- $controller = new TestController('test', $this->request);
- $this->reflector->reflect($controller, 'singleAttribute');
- $this->bruteForceMiddleware->afterController($controller, 'singleAttribute', $response);
- }
- public function testAfterControllerWithMultipleAttributesGeneralMatch(): void {
- /** @var Response|\PHPUnit\Framework\MockObject\MockObject $response */
- $response = $this->createMock(Response::class);
- $response
- ->expects($this->once())
- ->method('isThrottled')
- ->willReturn(true);
- $response
- ->expects($this->once())
- ->method('getThrottleMetadata')
- ->willReturn([]);
- $this->request
- ->expects($this->once())
- ->method('getRemoteAddress')
- ->willReturn('::1');
- $this->throttler
- ->expects($this->exactly(2))
- ->method('sleepDelayOrThrowOnMax')
- ->withConsecutive(
- ['::1', 'first'],
- ['::1', 'second'],
- );
- $this->throttler
- ->expects($this->exactly(2))
- ->method('registerAttempt')
- ->withConsecutive(
- ['first', '::1'],
- ['second', '::1'],
- );
- $controller = new TestController('test', $this->request);
- $this->reflector->reflect($controller, 'multipleAttributes');
- $this->bruteForceMiddleware->afterController($controller, 'multipleAttributes', $response);
- }
- public function testAfterControllerWithMultipleAttributesSpecificMatch(): void {
- /** @var Response|\PHPUnit\Framework\MockObject\MockObject $response */
- $response = $this->createMock(Response::class);
- $response
- ->expects($this->once())
- ->method('isThrottled')
- ->willReturn(true);
- $response
- ->expects($this->once())
- ->method('getThrottleMetadata')
- ->willReturn(['action' => 'second']);
- $this->request
- ->expects($this->once())
- ->method('getRemoteAddress')
- ->willReturn('::1');
- $this->throttler
- ->expects($this->once())
- ->method('sleepDelayOrThrowOnMax')
- ->with('::1', 'second');
- $this->throttler
- ->expects($this->once())
- ->method('registerAttempt')
- ->with('second', '::1');
- $controller = new TestController('test', $this->request);
- $this->reflector->reflect($controller, 'multipleAttributes');
- $this->bruteForceMiddleware->afterController($controller, 'multipleAttributes', $response);
- }
- public function testAfterControllerWithoutAnnotation(): void {
- $this->request
- ->expects($this->never())
- ->method('getRemoteAddress');
- $this->throttler
- ->expects($this->never())
- ->method('sleepDelayOrThrowOnMax');
- $controller = new TestController('test', $this->request);
- $this->reflector->reflect($controller, 'testMethodWithoutAnnotation');
- /** @var Response|\PHPUnit\Framework\MockObject\MockObject $response */
- $response = $this->createMock(Response::class);
- $this->bruteForceMiddleware->afterController($controller, 'testMethodWithoutAnnotation', $response);
- }
- public function testAfterControllerWithThrottledResponseButUnhandled(): void {
- $this->request
- ->expects($this->never())
- ->method('getRemoteAddress');
- $this->throttler
- ->expects($this->never())
- ->method('sleepDelayOrThrowOnMax');
- $controller = new TestController('test', $this->request);
- $this->reflector->reflect($controller, 'testMethodWithoutAnnotation');
- /** @var Response|\PHPUnit\Framework\MockObject\MockObject $response */
- $response = $this->createMock(Response::class);
- $response->method('isThrottled')
- ->willReturn(true);
- $this->logger->expects($this->once())
- ->method('debug')
- ->with('Response for Test\AppFramework\Middleware\Security\TestController::testMethodWithoutAnnotation got bruteforce throttled but has no annotation nor attribute defined.');
- $this->bruteForceMiddleware->afterController($controller, 'testMethodWithoutAnnotation', $response);
- }
- }
|