fix(rate-limit): remove 429 cooldown config option
This commit is contained in:
@@ -97,10 +97,9 @@ func TestHandle429_FallbackDisabledSkipsLocalMark(t *testing.T) {
|
||||
require.Zero(t, accountRepo.rateLimitCalls)
|
||||
}
|
||||
|
||||
func TestHandle429_FallbackUsesConfigSecondsWhenSettingServiceMissing(t *testing.T) {
|
||||
func TestHandle429_FallbackUsesDefaultSecondsWhenSettingServiceMissing(t *testing.T) {
|
||||
accountRepo := &rateLimit429AccountRepoStub{}
|
||||
cfg := &config.Config{}
|
||||
cfg.RateLimit.RateLimit429CooldownSeconds = 9
|
||||
svc := NewRateLimitService(accountRepo, nil, cfg, nil, nil)
|
||||
|
||||
account := &Account{ID: 44, Platform: PlatformGemini, Type: AccountTypeAPIKey}
|
||||
@@ -110,5 +109,5 @@ func TestHandle429_FallbackUsesConfigSecondsWhenSettingServiceMissing(t *testing
|
||||
|
||||
require.Equal(t, 1, accountRepo.rateLimitCalls)
|
||||
require.Equal(t, int64(44), accountRepo.lastRateLimitID)
|
||||
require.True(t, !accountRepo.lastRateLimitReset.Before(before.Add(9*time.Second)) && !accountRepo.lastRateLimitReset.After(after.Add(9*time.Second)))
|
||||
require.True(t, !accountRepo.lastRateLimitReset.Before(before.Add(5*time.Second)) && !accountRepo.lastRateLimitReset.After(after.Add(5*time.Second)))
|
||||
}
|
||||
|
||||
@@ -955,9 +955,6 @@ func (s *RateLimitService) get429FallbackCooldown(ctx context.Context, account *
|
||||
}
|
||||
|
||||
seconds := defaultRateLimit429CooldownSeconds
|
||||
if s.cfg != nil && s.cfg.RateLimit.RateLimit429CooldownSeconds > 0 {
|
||||
seconds = s.cfg.RateLimit.RateLimit429CooldownSeconds
|
||||
}
|
||||
seconds = clampRateLimit429CooldownSeconds(seconds)
|
||||
return time.Duration(seconds) * time.Second, true
|
||||
}
|
||||
|
||||
Reference in New Issue
Block a user