Skip to content

Commit e90e9ce

Browse files
ratelimits: Fix docs urls for two rate limits (#8596)
A previous website pull request (letsencrypt/website#1933) modified documentation anchor links so they no longer align with those referenced in Boulder error messages. Update documentation hyperlinks in Boulder errors to match the website changes: - https://letsencrypt.org/docs/rate-limits/#authorization-failures-per-identifier-per-account - https://letsencrypt.org/docs/rate-limits/#new-registrations-per-ip-address
1 parent 7113031 commit e90e9ce

2 files changed

Lines changed: 4 additions & 4 deletions

File tree

errors/errors.go

Lines changed: 2 additions & 2 deletions
Original file line numberDiff line numberDiff line change
@@ -253,15 +253,15 @@ func CertificatesPerFQDNSetError(retryAfter time.Duration, msg string, args ...a
253253
func FailedAuthorizationsPerDomainPerAccountError(retryAfter time.Duration, msg string, args ...any) error {
254254
return &BoulderError{
255255
Type: RateLimit,
256-
Detail: fmt.Sprintf(msg+": see https://letsencrypt.org/docs/rate-limits/#authorization-failures-per-hostname-per-account", args...),
256+
Detail: fmt.Sprintf(msg+": see https://letsencrypt.org/docs/rate-limits/#authorization-failures-per-identifier-per-account", args...),
257257
RetryAfter: retryAfter,
258258
}
259259
}
260260

261261
func LimitOverrideRequestsPerIPAddressError(retryAfter time.Duration, msg string, args ...any) error {
262262
return &BoulderError{
263263
Type: RateLimit,
264-
Detail: fmt.Sprintf(msg+": see https://letsencrypt.org/docs/rate-limits/#limit-override-requests-per-ip-address", args...),
264+
Detail: fmt.Sprintf(msg+": see https://letsencrypt.org/docs/rate-limits/#new-registrations-per-ip-address", args...),
265265
RetryAfter: retryAfter,
266266
}
267267
}

ratelimits/limiter_test.go

Lines changed: 2 additions & 2 deletions
Original file line numberDiff line numberDiff line change
@@ -527,7 +527,7 @@ func TestRateLimitError(t *testing.T) {
527527
bucketKey: "4:12345:example.com",
528528
},
529529
},
530-
expectedErr: "too many failed authorizations (7) for \"example.com\" in the last 1h0m0s, retry after 1970-01-01 00:00:15 UTC: see https://letsencrypt.org/docs/rate-limits/#authorization-failures-per-hostname-per-account",
530+
expectedErr: "too many failed authorizations (7) for \"example.com\" in the last 1h0m0s, retry after 1970-01-01 00:00:15 UTC: see https://letsencrypt.org/docs/rate-limits/#authorization-failures-per-identifier-per-account",
531531
expectedErrType: berrors.RateLimit,
532532
},
533533
{
@@ -577,7 +577,7 @@ func TestRateLimitError(t *testing.T) {
577577
},
578578
},
579579
},
580-
expectedErr: "too many override request form submissions (3) from this IP address in the last 1h0m0s, retry after 1970-01-01 00:00:20 UTC: see https://letsencrypt.org/docs/rate-limits/#limit-override-requests-per-ip-address",
580+
expectedErr: "too many override request form submissions (3) from this IP address in the last 1h0m0s, retry after 1970-01-01 00:00:20 UTC: see https://letsencrypt.org/docs/rate-limits/#new-registrations-per-ip-address",
581581
expectedErrType: berrors.RateLimit,
582582
},
583583
{

0 commit comments

Comments
 (0)