Retry-After returns with 0 seconds

Внести вклад Ошибка
1

When we retry after a 429, we get another 429.

I suspect a round-down issue.

In our scenario:

  • we do requests, and eventually get a 429 Too Many Request with Retry-After: 19
  • we wait 19 seconds, then try again:
    —> We get a 429 again, with Retry-After: 0

Now, can it be the actual values were 19.4 and 0.4 for example and they were rounded down for the Retry-After in the headers?

4 лайка
2

Probably, should be an easy fix. Another starter task @erlend_sh?

2 лайка
3

I’m not entirely sure where this is happening, but I suspect it has to do with use of Lua’s tonumber() in rate_limiter.rb—quoth Redis’ EVAL documentation:

Lua has a single numerical type, Lua numbers. There is no distinction between integers and floats. So we always convert Lua numbers into integer replies, removing the decimal part of the number if any.

4 лайка
4

Я занимаюсь этим багом и завтра отправлю pull-request.

1 лайк
5

Оказалось, что проблема не в округлении в Lua-скрипте. Lua-скрипт уже получает округлённые значения.

Более того, проблема вообще не в округлении. Могут использоваться неот округлённые секунды, и в любом случае можно получить ответ 429 с заголовком Retry-After: 0.

Вот исправление основной проблемы:

А вот исправление дополнительной проблемы, которая иногда может вызывать ту же ошибку:

3 лайка