Skip to content

Fix infinite DNS retry loop when IPv6 DNS servers are unreachable - #253

Merged
parfeon merged 2 commits into
masterfrom
CLEN-3630-fix-ipv6-ipv4-fallback
Jul 30, 2026
Merged

Fix infinite DNS retry loop when IPv6 DNS servers are unreachable#253
parfeon merged 2 commits into
masterfrom
CLEN-3630-fix-ipv6-ipv4-fallback

Conversation

@parfeon

@parfeon parfeon commented Jul 27, 2026

Copy link
Copy Markdown
Contributor

fix(dns): Fix infinite DNS retry loop when IPv6 DNS servers are unreachable

Clear the IPv6 address buffer in get_dns_ip() when a server is skipped due to prior failure marking, so user_provided_ipv6_dns correctly becomes false and the rotation falls through to IPv4 DNS servers.

…chable

Clear the IPv6 address buffer in `get_dns_ip()` when a server is skipped due to prior failure
marking, so `user_provided_ipv6_dns` correctly becomes false and the rotation falls through to IPv4
DNS servers.
@parfeon parfeon self-assigned this Jul 27, 2026
@parfeon
parfeon requested a review from Xavrax as a code owner July 27, 2026 21:05
@parfeon parfeon added the status: done This issue is considered resolved. label Jul 27, 2026
@parfeon
parfeon requested a review from KGronek-Pubnub as a code owner July 27, 2026 21:05
@parfeon parfeon added priority: high This PR should be reviewed ASAP. type: fix This PR contains fixes to existing features. labels Jul 27, 2026

@Xavrax Xavrax left a comment

Copy link
Copy Markdown
Contributor

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

LGTM

@parfeon

parfeon commented Jul 30, 2026

Copy link
Copy Markdown
Contributor Author

@pubnub-release-bot release

@parfeon
parfeon merged commit bc3a88a into master Jul 30, 2026
7 checks passed
@parfeon
parfeon deleted the CLEN-3630-fix-ipv6-ipv4-fallback branch July 30, 2026 19:08
@pubnub-release-bot

Copy link
Copy Markdown
Contributor

🚀 Release successfully completed 🚀

Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Labels

priority: high This PR should be reviewed ASAP. status: done This issue is considered resolved. type: fix This PR contains fixes to existing features.

Projects

None yet

Development

Successfully merging this pull request may close these issues.

3 participants