fix: replace hardcoded China DNS with global public DNS#20
Open
winstar0070 wants to merge 1 commit intomainfrom
Open
fix: replace hardcoded China DNS with global public DNS#20winstar0070 wants to merge 1 commit intomainfrom
winstar0070 wants to merge 1 commit intomainfrom
Conversation
Remove China-specific DNS servers (114.114.114.114, 119.29.29.29, 223.5.5.5) from new image initialization resolv.conf. Replace with Cloudflare (1.1.1.1) and Google (8.8.8.8, 8.8.4.4) as universal fallback DNS servers. Also fix a bug where /etc/resolv.conf was deleted before being backed up (rm before cp).
Member
Author
|
보류: DHCP/로컬 DNS 우선, 공개 DNS는 폴백으로 재설계 필요. udhcpc 성공 시 resolv.conf 비터치, 실패시에만 템플릿 복사. 기존 파일 보존 및 설정파일 기반 구성 주입 제안. |
Member
Author
|
Putting this PR on hold due to potential environment overrides. Summary: prefer DHCP/local resolv.conf; use public DNS only as fallback (udhcpc failure/AP mode); never overwrite existing resolv.conf (nameserver≥1); allow fallback via /etc/kvm/dns.conf; implement in S30eth/S30wifi + system_init; add safeguards/tests/docs. Will keep this PR as draft and follow up with split PRs. |
This file contains hidden or bidirectional Unicode text that may be interpreted or compiled differently than what appears below. To review, open the file in an editor that reveals hidden Unicode characters.
Learn more about bidirectional Unicode characters
Sign up for free
to join this conversation on GitHub.
Already have an account?
Sign in to comment
2 participants
Add this suggestion to a batch that can be applied as a single commit.This suggestion is invalid because no changes were made to the code.Suggestions cannot be applied while the pull request is closed.Suggestions cannot be applied while viewing a subset of changes.Only one suggestion per line can be applied in a batch.Add this suggestion to a batch that can be applied as a single commit.Applying suggestions on deleted lines is not supported.You must change the existing code in this line in order to create a valid suggestion.Outdated suggestions cannot be applied.This suggestion has been applied or marked resolved.Suggestions cannot be applied from pending reviews.Suggestions cannot be applied on multi-line comments.Suggestions cannot be applied while the pull request is queued to merge.Suggestion cannot be applied right now. Please check back later.
Summary
build_complete_resolv()rm -rf /etc/resolv.confthencp /etc/resolv.conf /etc/resolv.conf.old(backup after delete)kvm_new_imgflag)Addresses #1
Changed files
support/sg2002/kvm_system/main/lib/system_init/system_init.cppTest plan
/etc/resolv.confcontains only 1.1.1.1, 8.8.8.8, 8.8.4.4