[PR #675] [MERGED] Refactoring: Removing code duplication on the DPS Solvers #673

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

📋 Pull Request Information

Original PR: https://github.com/mageddo/dns-proxy-server/pull/675
Author: @mageddo
Created: 12/26/2025
Status: Merged
Merged: 12/26/2025
Merged by: @mageddo

Base: masterHead: feat/refactoring


📝 Commits (10+)

📊 Changes

99 files changed (+1144 additions, -727 deletions)

View changed files

📝 RELEASE-NOTES.md (+4 -0)
📝 gradle.properties (+1 -1)
📝 src/main/java/com/mageddo/dns/utils/Messages.java (+37 -22)
📝 src/main/java/com/mageddo/dnsproxyserver/config/Config.java (+17 -6)
📝 src/main/java/com/mageddo/dnsproxyserver/config/ConfigEntryTypes.java (+7 -17)
📝 src/main/java/com/mageddo/dnsproxyserver/di/Context.java (+2 -2)
📝 src/main/java/com/mageddo/dnsproxyserver/di/module/ModuleDao.java (+7 -9)
📝 src/main/java/com/mageddo/dnsproxyserver/di/module/ModuleMap.java (+6 -6)
📝 src/main/java/com/mageddo/dnsproxyserver/di/module/ModuleSolver.java (+6 -6)
📝 src/main/java/com/mageddo/dnsproxyserver/docker/ContainerDAO.java (+2 -2)
📝 src/main/java/com/mageddo/dnsproxyserver/docker/DockerNetworkDAO.java (+2 -3)
📝 src/main/java/com/mageddo/dnsproxyserver/docker/Drivers.java (+1 -1)
📝 src/main/java/com/mageddo/dnsproxyserver/docker/NetworkConnectionStatus.java (+1 -1)
📝 src/main/java/com/mageddo/dnsproxyserver/docker/application/ContainerPredicates.java (+1 -2)
📝 src/main/java/com/mageddo/dnsproxyserver/docker/dataprovider/ContainerDAOApi.java (+2 -1)
📝 src/main/java/com/mageddo/dnsproxyserver/docker/dataprovider/DockerNetworkDAOApi.java (+5 -4)
📝 src/main/java/com/mageddo/dnsproxyserver/quarkus/QuarkusConfig.java (+2 -2)
📝 src/main/java/com/mageddo/dnsproxyserver/server/dns/RequestHandlerDefault.java (+100 -42)
📝 src/main/java/com/mageddo/dnsproxyserver/server/rest/CacheResource.java (+2 -2)
📝 src/main/java/com/mageddo/dnsproxyserver/server/rest/reqres/CacheEntryResV1.java (+1 -1)

...and 79 more files

📄 Description

  • Removing code duplication on the DPS Solvers
  • Optimizing logs

🔄 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/675 **Author:** [@mageddo](https://github.com/mageddo) **Created:** 12/26/2025 **Status:** ✅ Merged **Merged:** 12/26/2025 **Merged by:** [@mageddo](https://github.com/mageddo) **Base:** `master` ← **Head:** `feat/refactoring` --- ### 📝 Commits (10+) - [`12bb0db`](https://github.com/mageddo/dns-proxy-server/commit/12bb0db14d5439f52de68bfaba93a7574d999107) adjusts - [`44d1797`](https://github.com/mageddo/dns-proxy-server/commit/44d179782ccc9823ef1037b6584ea5e52519b7ad) adjusts - [`aa78c85`](https://github.com/mageddo/dns-proxy-server/commit/aa78c8594c3463190a41c57a9235e9122d1e4d48) adjusts - [`54aca5f`](https://github.com/mageddo/dns-proxy-server/commit/54aca5f5e51a452922e7fdc2d792d685f16120d5) adjusts - [`69fcae9`](https://github.com/mageddo/dns-proxy-server/commit/69fcae9f1a73b3a1fa8072be3b18d808a0064bed) adjusts - [`17f62a4`](https://github.com/mageddo/dns-proxy-server/commit/17f62a4c4a694edcfdd8a1cf41ce842bb0b63670) adjusts - [`19b0f63`](https://github.com/mageddo/dns-proxy-server/commit/19b0f6387e920fff9fd56aed93bcc71697c9563c) adjusts - [`c9ad958`](https://github.com/mageddo/dns-proxy-server/commit/c9ad9583a016ef8a3b0a3d44c2ae5e869261ea33) adjusts - [`ad30a0f`](https://github.com/mageddo/dns-proxy-server/commit/ad30a0f7a8aea1bddcc7699ef6e0ff85007cf3ff) adjusts - [`36cc326`](https://github.com/mageddo/dns-proxy-server/commit/36cc326f236ac97f3aa3e8db684ac4791370eff5) adjusts ### 📊 Changes **99 files changed** (+1144 additions, -727 deletions) <details> <summary>View changed files</summary> 📝 `RELEASE-NOTES.md` (+4 -0) 📝 `gradle.properties` (+1 -1) 📝 `src/main/java/com/mageddo/dns/utils/Messages.java` (+37 -22) 📝 `src/main/java/com/mageddo/dnsproxyserver/config/Config.java` (+17 -6) 📝 `src/main/java/com/mageddo/dnsproxyserver/config/ConfigEntryTypes.java` (+7 -17) 📝 `src/main/java/com/mageddo/dnsproxyserver/di/Context.java` (+2 -2) 📝 `src/main/java/com/mageddo/dnsproxyserver/di/module/ModuleDao.java` (+7 -9) 📝 `src/main/java/com/mageddo/dnsproxyserver/di/module/ModuleMap.java` (+6 -6) 📝 `src/main/java/com/mageddo/dnsproxyserver/di/module/ModuleSolver.java` (+6 -6) 📝 `src/main/java/com/mageddo/dnsproxyserver/docker/ContainerDAO.java` (+2 -2) 📝 `src/main/java/com/mageddo/dnsproxyserver/docker/DockerNetworkDAO.java` (+2 -3) 📝 `src/main/java/com/mageddo/dnsproxyserver/docker/Drivers.java` (+1 -1) 📝 `src/main/java/com/mageddo/dnsproxyserver/docker/NetworkConnectionStatus.java` (+1 -1) 📝 `src/main/java/com/mageddo/dnsproxyserver/docker/application/ContainerPredicates.java` (+1 -2) 📝 `src/main/java/com/mageddo/dnsproxyserver/docker/dataprovider/ContainerDAOApi.java` (+2 -1) 📝 `src/main/java/com/mageddo/dnsproxyserver/docker/dataprovider/DockerNetworkDAOApi.java` (+5 -4) 📝 `src/main/java/com/mageddo/dnsproxyserver/quarkus/QuarkusConfig.java` (+2 -2) 📝 `src/main/java/com/mageddo/dnsproxyserver/server/dns/RequestHandlerDefault.java` (+100 -42) 📝 `src/main/java/com/mageddo/dnsproxyserver/server/rest/CacheResource.java` (+2 -2) 📝 `src/main/java/com/mageddo/dnsproxyserver/server/rest/reqres/CacheEntryResV1.java` (+1 -1) _...and 79 more files_ </details> ### 📄 Description * Removing code duplication on the DPS Solvers * Optimizing logs --- <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:58 +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#673
No description provided.