Skip to content

Commit

Permalink
ratelimit: Remove deprecated PendingOrdersPerAccount (#7053)
Browse files Browse the repository at this point in the history
Remove PendingOrdersPerAccount, which was officially deprecated in #3501
(5 years ago).

Part of #5545
  • Loading branch information
beautifulentropy authored Aug 28, 2023
1 parent 72e01b3 commit c8740c0
Show file tree
Hide file tree
Showing 2 changed files with 0 additions and 18 deletions.
5 changes: 0 additions & 5 deletions ra/ra_test.go
Original file line number Diff line number Diff line change
Expand Up @@ -237,7 +237,6 @@ type dummyRateLimitConfig struct {
RegistrationsPerIPPolicy ratelimit.RateLimitPolicy
RegistrationsPerIPRangePolicy ratelimit.RateLimitPolicy
PendingAuthorizationsPerAccountPolicy ratelimit.RateLimitPolicy
PendingOrdersPerAccountPolicy ratelimit.RateLimitPolicy
NewOrdersPerAccountPolicy ratelimit.RateLimitPolicy
InvalidAuthorizationsPerAccountPolicy ratelimit.RateLimitPolicy
CertificatesPerFQDNSetPolicy ratelimit.RateLimitPolicy
Expand All @@ -260,10 +259,6 @@ func (r *dummyRateLimitConfig) PendingAuthorizationsPerAccount() ratelimit.RateL
return r.PendingAuthorizationsPerAccountPolicy
}

func (r *dummyRateLimitConfig) PendingOrdersPerAccount() ratelimit.RateLimitPolicy {
return r.PendingOrdersPerAccountPolicy
}

func (r *dummyRateLimitConfig) NewOrdersPerAccount() ratelimit.RateLimitPolicy {
return r.NewOrdersPerAccountPolicy
}
Expand Down
13 changes: 0 additions & 13 deletions ratelimit/rate-limits.go
Original file line number Diff line number Diff line change
Expand Up @@ -18,7 +18,6 @@ type Limits interface {
InvalidAuthorizationsPerAccount() RateLimitPolicy
CertificatesPerFQDNSet() RateLimitPolicy
CertificatesPerFQDNSetFast() RateLimitPolicy
PendingOrdersPerAccount() RateLimitPolicy
NewOrdersPerAccount() RateLimitPolicy
LoadPolicies(contents []byte) error
}
Expand Down Expand Up @@ -95,15 +94,6 @@ func (r *limitsImpl) CertificatesPerFQDNSetFast() RateLimitPolicy {
return r.rlPolicy.CertificatesPerFQDNSetFast
}

func (r *limitsImpl) PendingOrdersPerAccount() RateLimitPolicy {
r.RLock()
defer r.RUnlock()
if r.rlPolicy == nil {
return RateLimitPolicy{}
}
return r.rlPolicy.PendingOrdersPerAccount
}

func (r *limitsImpl) NewOrdersPerAccount() RateLimitPolicy {
r.RLock()
defer r.RUnlock()
Expand Down Expand Up @@ -157,9 +147,6 @@ type rateLimitConfig struct {
// Note that this limit is actually "per account, per hostname," but that
// is too long for the variable name.
InvalidAuthorizationsPerAccount RateLimitPolicy `yaml:"invalidAuthorizationsPerAccount"`
// Number of pending orders that can exist per account. Overrides by key are
// not applied, but overrides by registration are. **DEPRECATED**
PendingOrdersPerAccount RateLimitPolicy `yaml:"pendingOrdersPerAccount"`
// Number of new orders that can be created per account within the given
// window. Overrides by key are not applied, but overrides by registration are.
NewOrdersPerAccount RateLimitPolicy `yaml:"newOrdersPerAccount"`
Expand Down

0 comments on commit c8740c0

Please sign in to comment.