[PR #447] [MERGED] Feat/440 refactoring #499

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

📋 Pull Request Information

Original PR: https://github.com/mageddo/dns-proxy-server/pull/447
Author: @mageddo
Created: 5/22/2024
Status: Merged
Merged: 5/23/2024
Merged by: @mageddo

Base: masterHead: feat/440-refactoring


📝 Commits (10+)

  • 855ba89 refactoring package and names
  • f5a4b74 refactoring package and names
  • a7cf25e isolation app from dataprovider
  • f3d972b refactrogin
  • c4b9c5d refactoring
  • 59847fa fixing compiling errors
  • 5fb0130 fixing compiling errors
  • b4b18af refactoring tests
  • 514b9dc creating dao strategy to find configs
  • 967be06 partially refactoring to build config based on multiple sources

📊 Changes

89 files changed (+1088 additions, -534 deletions)

View changed files

📝 RELEASE-NOTES.md (+3 -0)
📝 gradle.properties (+1 -1)
src/main/java/com/mageddo/di/CDIImpl.java (+62 -0)
📝 src/main/java/com/mageddo/di/InstanceImpl.java (+6 -1)
📝 src/main/java/com/mageddo/dnsproxyserver/App.java (+17 -3)
src/main/java/com/mageddo/dnsproxyserver/AppSettings.java (+0 -31)
src/main/java/com/mageddo/dnsproxyserver/application/AppSettings.java (+26 -0)
📝 src/main/java/com/mageddo/dnsproxyserver/config/Config.java (+12 -26)
📝 src/main/java/com/mageddo/dnsproxyserver/config/ConfigEntryTypes.java (+1 -1)
src/main/java/com/mageddo/dnsproxyserver/config/Configs.java (+0 -180)
📝 src/main/java/com/mageddo/dnsproxyserver/config/LogLevel.java (+1 -1)
src/main/java/com/mageddo/dnsproxyserver/config/application/ConfigService.java (+49 -0)
src/main/java/com/mageddo/dnsproxyserver/config/application/Configs.java (+31 -0)
src/main/java/com/mageddo/dnsproxyserver/config/configurator/Context.java (+19 -0)
src/main/java/com/mageddo/dnsproxyserver/config/configurator/module/ModuleConfigDAO.java (+31 -0)
src/main/java/com/mageddo/dnsproxyserver/config/dataprovider/ConfigDAO.java (+8 -0)
src/main/java/com/mageddo/dnsproxyserver/config/dataprovider/ConfigDAOCmdArgs.java (+60 -0)
src/main/java/com/mageddo/dnsproxyserver/config/dataprovider/ConfigDAOEnv.java (+47 -0)
src/main/java/com/mageddo/dnsproxyserver/config/dataprovider/ConfigDAOJson.java (+88 -0)
📝 src/main/java/com/mageddo/dnsproxyserver/config/dataprovider/ConfigPropDAO.java (+2 -2)

...and 69 more files

📄 Description

Purpose

Do refactoring found at #440 to be compliant to Hexagonal Arc and DDD evicting the problems and having the benefits it provides.


🔄 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/447 **Author:** [@mageddo](https://github.com/mageddo) **Created:** 5/22/2024 **Status:** ✅ Merged **Merged:** 5/23/2024 **Merged by:** [@mageddo](https://github.com/mageddo) **Base:** `master` ← **Head:** `feat/440-refactoring` --- ### 📝 Commits (10+) - [`855ba89`](https://github.com/mageddo/dns-proxy-server/commit/855ba89c5b4c4d871d92b63b663a119ae4ca72b5) refactoring package and names - [`f5a4b74`](https://github.com/mageddo/dns-proxy-server/commit/f5a4b74b4df5f9d23065b3142533097719965dcb) refactoring package and names - [`a7cf25e`](https://github.com/mageddo/dns-proxy-server/commit/a7cf25ead2ec62550d4f5eb727f2351060b66994) isolation app from dataprovider - [`f3d972b`](https://github.com/mageddo/dns-proxy-server/commit/f3d972bcf006548ca030d81b70b09cfe4351cb72) refactrogin - [`c4b9c5d`](https://github.com/mageddo/dns-proxy-server/commit/c4b9c5d08d882696e22ec48ee39e576b129b3407) refactoring - [`59847fa`](https://github.com/mageddo/dns-proxy-server/commit/59847facaa936c0e8c5a5d8ef2c1a40ec7505b42) fixing compiling errors - [`5fb0130`](https://github.com/mageddo/dns-proxy-server/commit/5fb01305ab62a114727ca99a270b594737469bf5) fixing compiling errors - [`b4b18af`](https://github.com/mageddo/dns-proxy-server/commit/b4b18af1045195f32146101ea88699bdb4c31bde) refactoring tests - [`514b9dc`](https://github.com/mageddo/dns-proxy-server/commit/514b9dc99dfb2763cc330498cfa6bd0ab9a918dd) creating dao strategy to find configs - [`967be06`](https://github.com/mageddo/dns-proxy-server/commit/967be06e1131b32c9aeaafc725ce640b9ffa99c4) partially refactoring to build config based on multiple sources ### 📊 Changes **89 files changed** (+1088 additions, -534 deletions) <details> <summary>View changed files</summary> 📝 `RELEASE-NOTES.md` (+3 -0) 📝 `gradle.properties` (+1 -1) ➕ `src/main/java/com/mageddo/di/CDIImpl.java` (+62 -0) 📝 `src/main/java/com/mageddo/di/InstanceImpl.java` (+6 -1) 📝 `src/main/java/com/mageddo/dnsproxyserver/App.java` (+17 -3) ➖ `src/main/java/com/mageddo/dnsproxyserver/AppSettings.java` (+0 -31) ➕ `src/main/java/com/mageddo/dnsproxyserver/application/AppSettings.java` (+26 -0) 📝 `src/main/java/com/mageddo/dnsproxyserver/config/Config.java` (+12 -26) 📝 `src/main/java/com/mageddo/dnsproxyserver/config/ConfigEntryTypes.java` (+1 -1) ➖ `src/main/java/com/mageddo/dnsproxyserver/config/Configs.java` (+0 -180) 📝 `src/main/java/com/mageddo/dnsproxyserver/config/LogLevel.java` (+1 -1) ➕ `src/main/java/com/mageddo/dnsproxyserver/config/application/ConfigService.java` (+49 -0) ➕ `src/main/java/com/mageddo/dnsproxyserver/config/application/Configs.java` (+31 -0) ➕ `src/main/java/com/mageddo/dnsproxyserver/config/configurator/Context.java` (+19 -0) ➕ `src/main/java/com/mageddo/dnsproxyserver/config/configurator/module/ModuleConfigDAO.java` (+31 -0) ➕ `src/main/java/com/mageddo/dnsproxyserver/config/dataprovider/ConfigDAO.java` (+8 -0) ➕ `src/main/java/com/mageddo/dnsproxyserver/config/dataprovider/ConfigDAOCmdArgs.java` (+60 -0) ➕ `src/main/java/com/mageddo/dnsproxyserver/config/dataprovider/ConfigDAOEnv.java` (+47 -0) ➕ `src/main/java/com/mageddo/dnsproxyserver/config/dataprovider/ConfigDAOJson.java` (+88 -0) 📝 `src/main/java/com/mageddo/dnsproxyserver/config/dataprovider/ConfigPropDAO.java` (+2 -2) _...and 69 more files_ </details> ### 📄 Description ## Purpose Do refactoring found at #440 to be compliant to [Hexagonal Arc](https://alistair.cockburn.us/hexagonal-architecture/) and [DDD](https://pt.slideshare.net/paucls/ddd-strategic-design-context-maps-paulo-clavijo-april-2018) evicting the problems and having the benefits it provides. --- <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:27 +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#499
No description provided.