[PR #191] [MERGED] checkout hosts before dns query #1295

New issue

Closed

opened 2026-03-16 01:58:37 +03:00 by kerem · 0 comments

kerem commented

2026-03-16 01:58:37 +03:00

Owner

📋 Pull Request Information

Original PR: https://github.com/hickory-dns/hickory-dns/pull/191
Author: @cssivision
Created: 9/19/2017
Status: ✅ Merged
Merged: 9/23/2017
Merged by: @bluejekyll

Base: master ← Head: master

📝 Commits (10+)

9702c3c checkout host before dns query
837f133 make Hosts and lookup pub
6c7de3f add integration test for lookup hosts
d3cc87b Merge branch 'master' of https://github.com/bluejekyll/trust-dns
a16a85c optimize some code
b82a53b add warnning when parse ip fail
37210db Merge branch 'master' of https://github.com/bluejekyll/trust-dns
e3ea900 Merge branch 'master' into master
86ec4c1 Merge branch 'master' into master
e7e5a5c Merge branch 'master' into master

📊 Changes

8 files changed (+307 additions, -7 deletions)

View changed files

📝 integration-tests/tests/lookup_tests.rs (+37 -2)
📝 resolver/src/config.rs (+3 -0)
➕ resolver/src/hosts.rs (+196 -0)
📝 resolver/src/lib.rs (+2 -0)
📝 resolver/src/lookup.rs (+2 -1)
📝 resolver/src/lookup_ip.rs (+17 -3)
📝 resolver/src/resolver_future.rs (+37 -1)
➕ resolver/tests/hosts (+13 -0)

📄 Description

this pr hasn't finished yet, but i have to make sure i'm in the right direction. whether it's necessary to reload check /etc/hosts file when this file changed, or just load this file at the start.

_{🔄 This issue represents a GitHub Pull Request. It cannot be merged through Gitea due to API limitations.}

## 📋 Pull Request Information **Original PR:** https://github.com/hickory-dns/hickory-dns/pull/191 **Author:** [@cssivision](https://github.com/cssivision) **Created:** 9/19/2017 **Status:** ✅ Merged **Merged:** 9/23/2017 **Merged by:** [@bluejekyll](https://github.com/bluejekyll) **Base:** `master` ← **Head:** `master` --- ### 📝 Commits (10+) - [`9702c3c`](https://github.com/hickory-dns/hickory-dns/commit/9702c3c85479ebfdd080097618f8a19bfed86630) checkout host before dns query - [`837f133`](https://github.com/hickory-dns/hickory-dns/commit/837f1338cdb38541b48ff32a2d6389b3812ad1e4) make Hosts and lookup pub - [`6c7de3f`](https://github.com/hickory-dns/hickory-dns/commit/6c7de3f6a04f4c1f1b0a481669ff065310a7a809) add integration test for lookup hosts - [`d3cc87b`](https://github.com/hickory-dns/hickory-dns/commit/d3cc87b0da9653064400972140316c621d9f0ffa) Merge branch 'master' of https://github.com/bluejekyll/trust-dns - [`a16a85c`](https://github.com/hickory-dns/hickory-dns/commit/a16a85c6c98f10d29ef43af5d2ed9824b3123be7) optimize some code - [`b82a53b`](https://github.com/hickory-dns/hickory-dns/commit/b82a53bb1904fe7f88a6be0429435685c2ae910a) add warnning when parse ip fail - [`37210db`](https://github.com/hickory-dns/hickory-dns/commit/37210dbf1e7ac2727224e7241857ae3b3002c131) Merge branch 'master' of https://github.com/bluejekyll/trust-dns - [`e3ea900`](https://github.com/hickory-dns/hickory-dns/commit/e3ea9005710339a8f110a711d3afbe994ca41569) Merge branch 'master' into master - [`86ec4c1`](https://github.com/hickory-dns/hickory-dns/commit/86ec4c19db77f16ff3501ca46c369923eb905364) Merge branch 'master' into master - [`e7e5a5c`](https://github.com/hickory-dns/hickory-dns/commit/e7e5a5c6b8451d410e8e515816d12ed418fb6ca7) Merge branch 'master' into master ### 📊 Changes **8 files changed** (+307 additions, -7 deletions) <details> <summary>View changed files</summary> 📝 `integration-tests/tests/lookup_tests.rs` (+37 -2) 📝 `resolver/src/config.rs` (+3 -0) ➕ `resolver/src/hosts.rs` (+196 -0) 📝 `resolver/src/lib.rs` (+2 -0) 📝 `resolver/src/lookup.rs` (+2 -1) 📝 `resolver/src/lookup_ip.rs` (+17 -3) 📝 `resolver/src/resolver_future.rs` (+37 -1) ➕ `resolver/tests/hosts` (+13 -0) </details> ### 📄 Description this pr hasn't finished yet, but i have to make sure i'm in the right direction. whether it's necessary to reload check `/etc/hosts` file when this file changed, or just load this file at the start. --- <sub>🔄 This issue represents a GitHub Pull Request. It cannot be merged through Gitea due to API limitations.</sub>