Skip to content

Commit

Permalink
chore: remove todo from auto tls (#2844)
Browse files Browse the repository at this point in the history
The `acme-client` module takes care of retries after 429s so no
need to handle it at this level.
  • Loading branch information
achingbrain authored Nov 26, 2024
1 parent b02ea9b commit 3650283
Showing 1 changed file with 0 additions and 2 deletions.
2 changes: 0 additions & 2 deletions packages/auto-tls/src/auto-tls.ts
Original file line number Diff line number Diff line change
Expand Up @@ -179,8 +179,6 @@ export class AutoTLS implements AutoTLSInterface {
private async fetchCertificate (multiaddrs: Multiaddr[], options?: AbortOptions): Promise<void> {
this.log('fetching certificate')

// TODO: handle rate limit errors like "too many new registrations (10) from this IP address in the last 3h0m0s, retry after 2024-11-01 09:22:38 UTC: see https://letsencrypt.org/docs/rate-limits/#new-registrations-per-ip-address"

const certificatePrivateKey = await loadOrCreateKey(this.keychain, this.certificatePrivateKeyName, this.certificatePrivateKeyBits)
const { pem, cert } = await this.loadOrCreateCertificate(certificatePrivateKey, multiaddrs, options)

Expand Down

0 comments on commit 3650283

Please sign in to comment.