[PR #322] [MERGED] Multiple resolvconf paths support #420

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

📋 Pull Request Information

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

Base: masterHead: systemd-resolved


📝 Commits (10+)

📊 Changes

22 files changed (+363 additions, -41 deletions)

View changed files

📝 RELEASE-NOTES.md (+9 -0)
📝 docker-compose-dev.yml (+6 -0)
📝 gradle.properties (+1 -1)
📝 src/main/java/com/mageddo/dnsproxyserver/config/Config.java (+1 -1)
📝 src/main/java/com/mageddo/dnsproxyserver/config/Configs.java (+2 -2)
📝 src/main/java/com/mageddo/dnsproxyserver/config/entrypoint/ConfigEnv.java (+3 -4)
📝 src/main/java/com/mageddo/dnsproxyserver/dnsconfigurator/DnsConfigurators.java (+3 -3)
📝 src/main/java/com/mageddo/dnsproxyserver/dnsconfigurator/linux/LinuxDnsConfigurator.java (+72 -8)
src/main/java/com/mageddo/dnsproxyserver/dnsconfigurator/linux/LinuxResolverConfDetector.java (+60 -0)
src/main/java/com/mageddo/dnsproxyserver/dnsconfigurator/linux/ResolvFile.java (+22 -0)
📝 src/main/java/com/mageddo/dnsproxyserver/dnsconfigurator/linux/resolvconf/DnsServerCleanerHandler.java (+1 -1)
📝 src/main/java/com/mageddo/dnsproxyserver/dnsconfigurator/linux/resolvconf/SetMachineDNSServerHandler.java (+1 -1)
📝 src/main/java/com/mageddo/dnsproxyserver/docker/DpsContainer.java (+2 -1)
📝 src/main/java/com/mageddo/dnsproxyserver/utils/Envs.java (+4 -0)
src/main/java/com/mageddo/dnsproxyserver/utils/Splits.java (+27 -0)
📝 src/main/java/com/mageddo/utils/Files.java (+17 -1)
📝 src/test/java/com/mageddo/dnsproxyserver/config/ConfigsTest.java (+1 -1)
📝 src/test/java/com/mageddo/dnsproxyserver/dnsconfigurator/linux/LinuxDnsConfiguratorTest.java (+51 -12)
src/test/java/com/mageddo/dnsproxyserver/dnsconfigurator/linux/LinuxResolverConfDetectorTest.java (+76 -0)
📝 src/test/java/com/mageddo/dnsproxyserver/templates/ConfigTemplates.java (+2 -1)

...and 2 more files

📄 Description

Based on #321 , it fixes

2 => MG_RESOLVCONF will now accept more than one value
3 => The default value of MG_RESOLVCONF will changes to ...


🔄 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/322 **Author:** [@mageddo](https://github.com/mageddo) **Created:** 2/25/2023 **Status:** ✅ Merged **Merged:** 2/25/2023 **Merged by:** [@mageddo](https://github.com/mageddo) **Base:** `master` ← **Head:** `systemd-resolved` --- ### 📝 Commits (10+) - [`3ee606a`](https://github.com/mageddo/dns-proxy-server/commit/3ee606a3c38c1c0f84520e704a6bf9a450468ddb) drafts, just drafts - [`0be7cb0`](https://github.com/mageddo/dns-proxy-server/commit/0be7cb0fc948ddf9348dce0ce001a2e354e0bd03) initial work to make it possible to pass multiple resolv conf files - [`430126b`](https://github.com/mageddo/dns-proxy-server/commit/430126b09a6e4628e9b52a6360b55e0452e32102) working to fix the tests - [`d217c3e`](https://github.com/mageddo/dns-proxy-server/commit/d217c3e60b530d00627bb78e01b80f7abbd3ecc7) creating type detector - [`cd268a6`](https://github.com/mageddo/dns-proxy-server/commit/cd268a6bff242111c5876153570b7e46d71ca515) tests were fixed - [`3be4321`](https://github.com/mageddo/dns-proxy-server/commit/3be4321aaa01090453be8a7e06962ebebc895b08) testing resolv conf decision - [`feb8e29`](https://github.com/mageddo/dns-proxy-server/commit/feb8e29d6ebe98e353db80812c77a7fcc61b9500) fixing tess - [`a5c1f1d`](https://github.com/mageddo/dns-proxy-server/commit/a5c1f1d32bfc0596dfc90201050002ac82cecc89) fixing tess - [`3788f22`](https://github.com/mageddo/dns-proxy-server/commit/3788f228b980e5c21afabe22c49c2e35cadfcf95) add volatile hint - [`575e703`](https://github.com/mageddo/dns-proxy-server/commit/575e7032195181900617bad3722d8893fb6b6811) systemd resolved detector ### 📊 Changes **22 files changed** (+363 additions, -41 deletions) <details> <summary>View changed files</summary> 📝 `RELEASE-NOTES.md` (+9 -0) 📝 `docker-compose-dev.yml` (+6 -0) 📝 `gradle.properties` (+1 -1) 📝 `src/main/java/com/mageddo/dnsproxyserver/config/Config.java` (+1 -1) 📝 `src/main/java/com/mageddo/dnsproxyserver/config/Configs.java` (+2 -2) 📝 `src/main/java/com/mageddo/dnsproxyserver/config/entrypoint/ConfigEnv.java` (+3 -4) 📝 `src/main/java/com/mageddo/dnsproxyserver/dnsconfigurator/DnsConfigurators.java` (+3 -3) 📝 `src/main/java/com/mageddo/dnsproxyserver/dnsconfigurator/linux/LinuxDnsConfigurator.java` (+72 -8) ➕ `src/main/java/com/mageddo/dnsproxyserver/dnsconfigurator/linux/LinuxResolverConfDetector.java` (+60 -0) ➕ `src/main/java/com/mageddo/dnsproxyserver/dnsconfigurator/linux/ResolvFile.java` (+22 -0) 📝 `src/main/java/com/mageddo/dnsproxyserver/dnsconfigurator/linux/resolvconf/DnsServerCleanerHandler.java` (+1 -1) 📝 `src/main/java/com/mageddo/dnsproxyserver/dnsconfigurator/linux/resolvconf/SetMachineDNSServerHandler.java` (+1 -1) 📝 `src/main/java/com/mageddo/dnsproxyserver/docker/DpsContainer.java` (+2 -1) 📝 `src/main/java/com/mageddo/dnsproxyserver/utils/Envs.java` (+4 -0) ➕ `src/main/java/com/mageddo/dnsproxyserver/utils/Splits.java` (+27 -0) 📝 `src/main/java/com/mageddo/utils/Files.java` (+17 -1) 📝 `src/test/java/com/mageddo/dnsproxyserver/config/ConfigsTest.java` (+1 -1) 📝 `src/test/java/com/mageddo/dnsproxyserver/dnsconfigurator/linux/LinuxDnsConfiguratorTest.java` (+51 -12) ➕ `src/test/java/com/mageddo/dnsproxyserver/dnsconfigurator/linux/LinuxResolverConfDetectorTest.java` (+76 -0) 📝 `src/test/java/com/mageddo/dnsproxyserver/templates/ConfigTemplates.java` (+2 -1) _...and 2 more files_ </details> ### 📄 Description Based on #321 , it fixes > 2 => MG_RESOLVCONF will now accept more than one value > 3 => The default value of MG_RESOLVCONF will changes to ... --- <sub>🔄 This issue represents a GitHub Pull Request. It cannot be merged through Gitea due to API limitations.</sub>
kerem 2026-02-26 04:35:11 +03:00
Sign in to join this conversation.
No milestone
No project
No assignees
1 participant
Notifications
Due date
The due date is invalid or out of range. Please use the format "yyyy-mm-dd".

No due date set.

Dependencies

No dependencies set.

Reference
starred/dns-proxy-server-mageddo#420
No description provided.