[PR #312] [MERGED] Respecting OS to configure as default DNS #413

New issue

Closed

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

kerem commented

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

Owner

📋 Pull Request Information

Original PR: https://github.com/mageddo/dns-proxy-server/pull/312
Author: @mageddo
Created: 2/24/2023
Status: ✅ Merged
Merged: 2/24/2023
Merged by: @mageddo

Base: master ← Head: feat/308

📝 Commits (7)

9056039 Resolvconf is a linux specific configuration can't be triggered at windows
4e5d67a Adjusting npe
e696b58 creating singletons
408067c is not possible to create a singleton for different classloaders
b1b5330 doc reference
9fc554a singleton for configurator
ac40a75 release notes

📊 Changes

10 files changed (+154 additions, -46 deletions)

View changed files

📝 RELEASE-NOTES.md (+4 -0)
➕ src/main/java/com/mageddo/commons/lang/Singletons.java (+60 -0)
📝 src/main/java/com/mageddo/dnsproxyserver/App.java (+1 -1)
📝 src/main/java/com/mageddo/dnsproxyserver/config/Configs.java (+30 -22)
📝 src/main/java/com/mageddo/dnsproxyserver/dnsconfigurator/DnsConfigurator.java (+2 -2)
📝 src/main/java/com/mageddo/dnsproxyserver/dnsconfigurator/DnsConfigurators.java (+20 -6)
📝 src/main/java/com/mageddo/dnsproxyserver/dnsconfigurator/linux/LinuxDnsConfigurator.java (+8 -3)
📝 src/test/java/com/mageddo/dnsproxyserver/config/ConfigsTest.java (+3 -3)
📝 src/test/java/com/mageddo/dnsproxyserver/dnsconfigurator/linux/LinuxDnsConfiguratorTest.java (+24 -7)
📝 src/test/java/com/mageddo/dnsproxyserver/docker/ContainerHostnameMatcherTest.java (+2 -2)

📄 Description

Partially fixing #308

_{🔄 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/312 **Author:** [@mageddo](https://github.com/mageddo) **Created:** 2/24/2023 **Status:** ✅ Merged **Merged:** 2/24/2023 **Merged by:** [@mageddo](https://github.com/mageddo) **Base:** `master` ← **Head:** `feat/308` --- ### 📝 Commits (7) - [`9056039`](https://github.com/mageddo/dns-proxy-server/commit/905603908b441de04004a07973700a552858c544) Resolvconf is a linux specific configuration can't be triggered at windows - [`4e5d67a`](https://github.com/mageddo/dns-proxy-server/commit/4e5d67a41d7ccdd6a2253879398a97ebf93ba4ce) Adjusting npe - [`e696b58`](https://github.com/mageddo/dns-proxy-server/commit/e696b5841e3c8471b5f0e89f6b44fcd2ab49799b) creating singletons - [`408067c`](https://github.com/mageddo/dns-proxy-server/commit/408067cdd414c237ada62a8429114b50bf7e6446) is not possible to create a singleton for different classloaders - [`b1b5330`](https://github.com/mageddo/dns-proxy-server/commit/b1b533023f06f6a183633d7161754a6d6b23b306) doc reference - [`9fc554a`](https://github.com/mageddo/dns-proxy-server/commit/9fc554a53910f796c0936bb9da00f7810aa7693f) singleton for configurator - [`ac40a75`](https://github.com/mageddo/dns-proxy-server/commit/ac40a75359e22dcadbccd8e5e52d3f6f76cdc92e) release notes ### 📊 Changes **10 files changed** (+154 additions, -46 deletions) <details> <summary>View changed files</summary> 📝 `RELEASE-NOTES.md` (+4 -0) ➕ `src/main/java/com/mageddo/commons/lang/Singletons.java` (+60 -0) 📝 `src/main/java/com/mageddo/dnsproxyserver/App.java` (+1 -1) 📝 `src/main/java/com/mageddo/dnsproxyserver/config/Configs.java` (+30 -22) 📝 `src/main/java/com/mageddo/dnsproxyserver/dnsconfigurator/DnsConfigurator.java` (+2 -2) 📝 `src/main/java/com/mageddo/dnsproxyserver/dnsconfigurator/DnsConfigurators.java` (+20 -6) 📝 `src/main/java/com/mageddo/dnsproxyserver/dnsconfigurator/linux/LinuxDnsConfigurator.java` (+8 -3) 📝 `src/test/java/com/mageddo/dnsproxyserver/config/ConfigsTest.java` (+3 -3) 📝 `src/test/java/com/mageddo/dnsproxyserver/dnsconfigurator/linux/LinuxDnsConfiguratorTest.java` (+24 -7) 📝 `src/test/java/com/mageddo/dnsproxyserver/docker/ContainerHostnameMatcherTest.java` (+2 -2) </details> ### 📄 Description Partially fixing #308 --- <sub>🔄 This issue represents a GitHub Pull Request. It cannot be merged through Gitea due to API limitations.</sub>