laravel - throttle 时间问题

Question

我在 trait ThrottlesLogins 中添加了以下方法在 Laravel 5.5 中

protected function TotalRegisterAttemptsLeft($request) {
    $this->incrementAttempts($request);
    return $this->limiter()->retriesLeft($this->resolveRequestSignature($request), 3);
}

路线

Route::post('apiregister', 
    array(
        'uses'          =>  'API\Register\RegisterAPIController@Registration', 
        'as'            =>  'apiRegister',
        'middleware'    =>  'throttle:3,1'
    )
);

这种方法在 5.4 中运行良好，让我解释一下问题。

我有一个最多 3 次尝试的注册 POST 路由。使用所有三个尝试后，用户将必须等待 60 秒。

但问题是，假设我在 3 次尝试中花费了 12 秒。尝试 3 次后，它说，请在 48 秒后尝试。而是应该说，请在 60 秒后重试。

如果您需要更多详细信息，请告诉我。

Answer 1

这是 ThrottleRequests-Middleware 中的错误或预期行为。在更改现有集成测试时，您可以轻松重现这一点:

public function test_lock_opens_immediately_after_decay()
{
    Carbon::setTestNow(null);

    Route::get('/', function () {
        return 'yes';
    })->middleware(ThrottleRequests::class.':2,1');

    $response = $this->withoutExceptionHandling()->get('/');
    $this->assertEquals('yes', $response->getContent());
    $this->assertEquals(2, $response->headers->get('X-RateLimit-Limit'));
    $this->assertEquals(1, $response->headers->get('X-RateLimit-Remaining'));

    Carbon::setTestNow(
        Carbon::now()->addSeconds(10)
    );

    $response = $this->withoutExceptionHandling()->get('/');
    $this->assertEquals('yes', $response->getContent());
    $this->assertEquals(2, $response->headers->get('X-RateLimit-Limit'));
    $this->assertEquals(0, $response->headers->get('X-RateLimit-Remaining'));

    Carbon::setTestNow(
        Carbon::now()->addSeconds(58)
    );

    try {
        $this->withoutExceptionHandling()->get('/');
    } catch (Throwable $e) {
        $this->assertEquals(429, $e->getStatusCode());
        $this->assertEquals(2, $e->getHeaders()['X-RateLimit-Limit']);
        $this->assertEquals(0, $e->getHeaders()['X-RateLimit-Remaining']);
        $this->assertEquals(2, $e->getHeaders()['Retry-After']);
        $this->assertEquals(Carbon::now()->addSeconds(2)->getTimestamp(), $e->getHeaders()['X-RateLimit-Reset']);
    }
}

我刚加

Carbon::setTestNow(
    Carbon::now()->addSeconds(10)
);

在第一个和第二个请求之间。这将导致来自 phpunit 的以下输出:

./vendor/bin/phpunit tests/Integration/Http/ThrottleRequestsTest.php
PHPUnit 6.5.5 by Sebastian Bergmann and contributors.

Runtime:       PHP 7.2.1
Configuration: /Volumes/Workspace/Projects/laravel/phpunit.xml.dist

F                                                                   1 / 1 (100%)

Time: 172 ms, Memory: 10.00MB

There was 1 failure:

1) Illuminate\Tests\Integration\Http\ThrottleRequestsTest::test_lock_opens_immediately_after_decay
Failed asserting that -8 matches expected 2.

/Volumes/Workspace/Projects/laravel/tests/Integration/Http/ThrottleRequestsTest.php:54

我用失败的测试创建了一个 PR ~一旦确认这不是预期的行为，有人可以开始修复它:~

https://github.com/laravel/framework/pull/22725/files

编辑:如 PR 中所述，这是预期行为。中间件专为 API 速率限制而设计，您希望确保在特定时间范围内允许 x 数量的请求通过。为了限制密码尝试，您必须切换底层 RateLimiter。

至于为什么这以前有效，我不能告诉你。