[PR #43] [MERGED] Local Resolution Optimizations #241

New issue

Closed

opened 2026-02-26 04:34:35 +03:00 by kerem · 0 comments

kerem commented

2026-02-26 04:34:35 +03:00

Owner

📋 Pull Request Information

Original PR: https://github.com/mageddo/dns-proxy-server/pull/43
Author: @mageddo
Created: 10/1/2017
Status: ✅ Merged
Merged: 10/14/2017
Merged by: @mageddo

Base: master ← Head: feature/42

📝 Commits (10+)

89cf529 #42 remove unnecessary save, defer function encapsulation and adding save time
3dbcaa0 #42 creating cache
d1cfeac #42 implementing size limitation
63ecb3d #42 LRU implemented
2d1c3d0 #42 using third part lru impl
bf53e55 #42 using cache at local events
a593855 #42 adding lru implementation
efd3c5f #42 fixing millis compare
b44a91b #42 fixing compile errors
5788309 #42 decrease log levelsm fixing nil pointer and adding stack info

📊 Changes

170 files changed (+143426 additions, -275 deletions)

View changed files

📝 RELEASE-NOTES.md (+5 -0)
📝 VERSION (+1 -1)
📝 builder (+1 -1)
➕ cache/cache.go (+12 -0)
➕ cache/lru/hashicorp/hashicorp.go (+16 -0)
➕ cache/lru/lru.go (+61 -0)
➕ cache/lru/lru_test.go (+85 -0)
➕ cache/store/store.go (+18 -0)
➕ cache/timed/timedvalue.go (+44 -0)
➕ cache/timed/timedvalue_test.go (+23 -0)
📝 conf/conf.go (+13 -21)
📝 conf/conf_test.go (+35 -1)
📝 controller/env.go (+53 -24)
➕ controller/env_test.go (+147 -0)
📝 controller/hostname.go (+60 -33)
➕ controller/hostname_test.go (+177 -0)
📝 controller/index.go (+6 -2)
📝 controller/main.go (+1 -0)
📝 dns.go (+11 -9)
📝 docker-compose.yml (+1 -1)

...and 80 more files

📄 Description

Increased code coverage
Implementing cache at local hostnames and remote server resolution
Considering TTL to invalidate hostname cache for local resolution

Fixing #42

_{🔄 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/mageddo/dns-proxy-server/pull/43 **Author:** [@mageddo](https://github.com/mageddo) **Created:** 10/1/2017 **Status:** ✅ Merged **Merged:** 10/14/2017 **Merged by:** [@mageddo](https://github.com/mageddo) **Base:** `master` ← **Head:** `feature/42` --- ### 📝 Commits (10+) - [`89cf529`](https://github.com/mageddo/dns-proxy-server/commit/89cf5296ff1484f2666c1c10681a7ccc7d7b7dc0) #42 remove unnecessary save, defer function encapsulation and adding save time - [`3dbcaa0`](https://github.com/mageddo/dns-proxy-server/commit/3dbcaa0316eb05fe25b3ed55d13165b8eba725f2) #42 creating cache - [`d1cfeac`](https://github.com/mageddo/dns-proxy-server/commit/d1cfeac6629f4349ad262607810ad6e4e8ce77ba) #42 implementing size limitation - [`63ecb3d`](https://github.com/mageddo/dns-proxy-server/commit/63ecb3d20db8800517a193b1ab99b2dafb992174) #42 LRU implemented - [`2d1c3d0`](https://github.com/mageddo/dns-proxy-server/commit/2d1c3d0410c9feee00909b231c5d272f9c2b4151) #42 using third part lru impl - [`bf53e55`](https://github.com/mageddo/dns-proxy-server/commit/bf53e5569fee5719279ed621e7e6ae21ee3a67c5) #42 using cache at local events - [`a593855`](https://github.com/mageddo/dns-proxy-server/commit/a5938557152b0b47edfad562b690281dbb1cc15a) #42 adding lru implementation - [`efd3c5f`](https://github.com/mageddo/dns-proxy-server/commit/efd3c5fa86ff6066b5901586c29a591dfd4bb23a) #42 fixing millis compare - [`b44a91b`](https://github.com/mageddo/dns-proxy-server/commit/b44a91b520eea2277bcff20e683b24d0b787837c) #42 fixing compile errors - [`5788309`](https://github.com/mageddo/dns-proxy-server/commit/5788309ecfe8cb44f9c6eae0be7816e8054ed8d9) #42 decrease log levelsm fixing nil pointer and adding stack info ### 📊 Changes **170 files changed** (+143426 additions, -275 deletions) <details> <summary>View changed files</summary> 📝 `RELEASE-NOTES.md` (+5 -0) 📝 `VERSION` (+1 -1) 📝 `builder` (+1 -1) ➕ `cache/cache.go` (+12 -0) ➕ `cache/lru/hashicorp/hashicorp.go` (+16 -0) ➕ `cache/lru/lru.go` (+61 -0) ➕ `cache/lru/lru_test.go` (+85 -0) ➕ `cache/store/store.go` (+18 -0) ➕ `cache/timed/timedvalue.go` (+44 -0) ➕ `cache/timed/timedvalue_test.go` (+23 -0) 📝 `conf/conf.go` (+13 -21) 📝 `conf/conf_test.go` (+35 -1) 📝 `controller/env.go` (+53 -24) ➕ `controller/env_test.go` (+147 -0) 📝 `controller/hostname.go` (+60 -33) ➕ `controller/hostname_test.go` (+177 -0) 📝 `controller/index.go` (+6 -2) 📝 `controller/main.go` (+1 -0) 📝 `dns.go` (+11 -9) 📝 `docker-compose.yml` (+1 -1) _...and 80 more files_ </details> ### 📄 Description * Increased code coverage * Implementing cache at local hostnames and remote server resolution * Considering TTL to invalidate hostname cache for local resolution Fixing #42 --- <sub>🔄 This issue represents a GitHub Pull Request. It cannot be merged through Gitea due to API limitations.</sub>