[PR #622] [MERGED] Refactoring | Create packages for each config data format module #627

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

📋 Pull Request Information

Original PR: https://github.com/mageddo/dns-proxy-server/pull/622
Author: @mageddo
Created: 3/31/2025
Status: Merged
Merged: 3/31/2025
Merged by: @mageddo

Base: masterHead: feat/594-dynamic-factory


📝 Commits (10+)

📊 Changes

60 files changed (+213 additions, -248 deletions)

View changed files

📝 RELEASE-NOTES.md (+3 -0)
📝 gradle.properties (+1 -1)
📝 src/main/java/com/mageddo/dnsproxyserver/App.java (+2 -2)
📝 src/main/java/com/mageddo/dnsproxyserver/config/Config.java (+12 -18)
📝 src/main/java/com/mageddo/dnsproxyserver/config/application/ConfigFileFinderService.java (+5 -5)
📝 src/main/java/com/mageddo/dnsproxyserver/config/application/ConfigPathMapper.java (+1 -1)
📝 src/main/java/com/mageddo/dnsproxyserver/config/application/ConfigService.java (+4 -2)
src/main/java/com/mageddo/dnsproxyserver/config/configurator/Context.java (+0 -19)
src/main/java/com/mageddo/dnsproxyserver/config/configurator/module/ModuleConfigDAO.java (+0 -31)
src/main/java/com/mageddo/dnsproxyserver/config/dataprovider/ConfigPropDAO.java (+0 -15)
src/main/java/com/mageddo/dnsproxyserver/config/dataprovider/VersionDAO.java (+5 -0)
📝 src/main/java/com/mageddo/dnsproxyserver/config/di/module/ModuleConfigDAO.java (+11 -4)
📝 src/main/java/com/mageddo/dnsproxyserver/config/mapper/ConfigMapper.java (+14 -16)
📝 src/main/java/com/mageddo/dnsproxyserver/config/mapper/LogLevelMapper.java (+2 -2)
📝 src/main/java/com/mageddo/dnsproxyserver/config/provider/cmdargs/ConfigDAOCmdArgs.java (+4 -3)
📝 src/main/java/com/mageddo/dnsproxyserver/config/provider/cmdargs/mapper/ConfigFlagMapper.java (+4 -3)
📝 src/main/java/com/mageddo/dnsproxyserver/config/provider/cmdargs/vo/ConfigFlag.java (+8 -4)
📝 src/main/java/com/mageddo/dnsproxyserver/config/provider/cmdargs/vo/package-info.java (+2 -2)
📝 src/main/java/com/mageddo/dnsproxyserver/config/provider/jsonv1v2/dataprovider/ConfigDAOJson.java (+3 -2)
📝 src/main/java/com/mageddo/dnsproxyserver/config/provider/jsonv1v2/dataprovider/JsonConfigs.java (+7 -7)

...and 40 more files

📄 Description

No description provided


🔄 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/622 **Author:** [@mageddo](https://github.com/mageddo) **Created:** 3/31/2025 **Status:** ✅ Merged **Merged:** 3/31/2025 **Merged by:** [@mageddo](https://github.com/mageddo) **Base:** `master` ← **Head:** `feat/594-dynamic-factory` --- ### 📝 Commits (10+) - [`401eba7`](https://github.com/mageddo/dns-proxy-server/commit/401eba73a5e5231e7fe709dd2dc6982c98e6fe6a) clean code - [`1a821b3`](https://github.com/mageddo/dns-proxy-server/commit/1a821b3b0f00014aa5d6579ee77fbd0f00ad1390) the current env will be refactored to legacy env - [`c11e6a3`](https://github.com/mageddo/dns-proxy-server/commit/c11e6a3675b9a6e5df5673b7e54498d9bacf3ee8) refactoring packages to clean arch - [`bfd06c2`](https://github.com/mageddo/dns-proxy-server/commit/bfd06c2368cc7c7c3954b8ca83975f95744de88f) refactoring - [`923a533`](https://github.com/mageddo/dns-proxy-server/commit/923a53368485fabeb8ef24f956bf519598076133) refactoring - [`db6ff94`](https://github.com/mageddo/dns-proxy-server/commit/db6ff941127f31984e235059d549e78a5dd9c3e0) refactoring - [`bb83c1a`](https://github.com/mageddo/dns-proxy-server/commit/bb83c1ab6b844153a33c39be1d9d2a9c10d1593c) refactoring - [`216ab27`](https://github.com/mageddo/dns-proxy-server/commit/216ab27a8f5adcb52f0d5ac89f0f617d4d6d00c3) refactoring - [`af426d7`](https://github.com/mageddo/dns-proxy-server/commit/af426d755f364df89a14ecbd4ecf12b0ef49d448) fixing test - [`08a902b`](https://github.com/mageddo/dns-proxy-server/commit/08a902b8b1071ce4bcd76366ad0242bfcf3ebd85) fixing reflection config ### 📊 Changes **60 files changed** (+213 additions, -248 deletions) <details> <summary>View changed files</summary> 📝 `RELEASE-NOTES.md` (+3 -0) 📝 `gradle.properties` (+1 -1) 📝 `src/main/java/com/mageddo/dnsproxyserver/App.java` (+2 -2) 📝 `src/main/java/com/mageddo/dnsproxyserver/config/Config.java` (+12 -18) 📝 `src/main/java/com/mageddo/dnsproxyserver/config/application/ConfigFileFinderService.java` (+5 -5) 📝 `src/main/java/com/mageddo/dnsproxyserver/config/application/ConfigPathMapper.java` (+1 -1) 📝 `src/main/java/com/mageddo/dnsproxyserver/config/application/ConfigService.java` (+4 -2) ➖ `src/main/java/com/mageddo/dnsproxyserver/config/configurator/Context.java` (+0 -19) ➖ `src/main/java/com/mageddo/dnsproxyserver/config/configurator/module/ModuleConfigDAO.java` (+0 -31) ➖ `src/main/java/com/mageddo/dnsproxyserver/config/dataprovider/ConfigPropDAO.java` (+0 -15) ➕ `src/main/java/com/mageddo/dnsproxyserver/config/dataprovider/VersionDAO.java` (+5 -0) 📝 `src/main/java/com/mageddo/dnsproxyserver/config/di/module/ModuleConfigDAO.java` (+11 -4) 📝 `src/main/java/com/mageddo/dnsproxyserver/config/mapper/ConfigMapper.java` (+14 -16) 📝 `src/main/java/com/mageddo/dnsproxyserver/config/mapper/LogLevelMapper.java` (+2 -2) 📝 `src/main/java/com/mageddo/dnsproxyserver/config/provider/cmdargs/ConfigDAOCmdArgs.java` (+4 -3) 📝 `src/main/java/com/mageddo/dnsproxyserver/config/provider/cmdargs/mapper/ConfigFlagMapper.java` (+4 -3) 📝 `src/main/java/com/mageddo/dnsproxyserver/config/provider/cmdargs/vo/ConfigFlag.java` (+8 -4) 📝 `src/main/java/com/mageddo/dnsproxyserver/config/provider/cmdargs/vo/package-info.java` (+2 -2) 📝 `src/main/java/com/mageddo/dnsproxyserver/config/provider/jsonv1v2/dataprovider/ConfigDAOJson.java` (+3 -2) 📝 `src/main/java/com/mageddo/dnsproxyserver/config/provider/jsonv1v2/dataprovider/JsonConfigs.java` (+7 -7) _...and 40 more files_ </details> ### 📄 Description _No description provided_ --- <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:50 +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#627
No description provided.