Skip to content

feat: allow to bypass DNS TTL entirely#1031

Merged
levkk merged 3 commits into
mainfrom
levkk-dns-cache-change
Jun 4, 2026
Merged

feat: allow to bypass DNS TTL entirely#1031
levkk merged 3 commits into
mainfrom
levkk-dns-cache-change

Conversation

@levkk

@levkk levkk commented Jun 4, 2026

Copy link
Copy Markdown
Collaborator

Allow dns_ttl to be set to 0, ignoring TTL from the name servers entirely. This wasn't possible previously because our DNS cache was implemented as a background loop, which would of DoS'ed nameservers.

@codecov

codecov Bot commented Jun 4, 2026

Copy link
Copy Markdown

Codecov Report

❌ Patch coverage is 97.00000% with 3 lines in your changes missing coverage. Please review.

Files with missing lines Patch % Lines
pgdog/src/backend/pool/dns_cache.rs 96.29% 3 Missing ⚠️

📢 Thoughts on this report? Let us know!

@levkk
levkk merged commit f016ce9 into main Jun 4, 2026
23 of 24 checks passed
@levkk
levkk deleted the levkk-dns-cache-change branch June 4, 2026 17:20
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Labels

None yet

Projects

None yet

Development

Successfully merging this pull request may close these issues.

1 participant