starred/dns-proxy-server-mageddo
1
0
Fork 0
mirror of https://github.com/mageddo/dns-proxy-server.git synced 2026-04-25 09:25:56 +03:00
Code Issues 10 Projects Releases 10 Packages Wiki Activity

[PR #445] [MERGED] Do Refactoring encountered at #442 #498

New issue
Closed
opened 2026-02-26 04:35:26 +03:00 by kerem · 0 comments
kerem commented 2026-02-26 04:35:26 +03:00
Owner
Copy link

📋 Pull Request Information

Original PR: https://github.com/mageddo/dns-proxy-server/pull/445
Author: @mageddo
Created: 5/13/2024
Status: ✅ Merged
Merged: 5/13/2024
Merged by: @mageddo

Base: master ← Head: feat/444

📝 Commits (10+)

  • 16c7bae refactoring unnecessary parameter
  • f7adfe8 refactoring class packages
  • 284a538 isolating container solving in app layer
  • c4d9023 a lot of refactoring trying to separate solver.docker module application from dataprovider impl
  • 37c37ca app compiling errors fixed
  • 0d4ba7f testing container mapper
  • d10ca10 mustSolveSpecifiedNetworkFirst fixed
  • 00f8180 refactoring test location
  • 3d4f23f new test
  • a41c588 creating new test

📊 Changes

86 files changed (+2027 additions, -994 deletions)

View changed files

📝 src/main/java/com/mageddo/dnsproxyserver/config/Config.java (+1 -1)
📝 src/main/java/com/mageddo/dnsproxyserver/config/ConfigDAO.java (+2 -0)
📝 src/main/java/com/mageddo/dnsproxyserver/config/ConfigDAOJson.java (+5 -0)
📝 src/main/java/com/mageddo/dnsproxyserver/config/Configs.java (+1 -1)
📝 src/main/java/com/mageddo/dnsproxyserver/di/Context.java (+4 -7)
📝 src/main/java/com/mageddo/dnsproxyserver/di/module/ModuleDao.java (+29 -8)
📝 src/main/java/com/mageddo/dnsproxyserver/di/module/ModuleMap.java (+12 -12)
📝 src/main/java/com/mageddo/dnsproxyserver/di/module/ModuleStartup.java (+1 -1)
📝 src/main/java/com/mageddo/dnsproxyserver/dnsconfigurator/DnsConfigurators.java (+3 -2)
➖ src/main/java/com/mageddo/dnsproxyserver/dnsconfigurator/DpsIpDiscover.java (+0 -32)
➖ src/main/java/com/mageddo/dnsproxyserver/docker/ContainerDAO.java (+0 -12)
➖ src/main/java/com/mageddo/dnsproxyserver/docker/ContainerSolvingService.java (+0 -180)
➖ src/main/java/com/mageddo/dnsproxyserver/docker/DockerDAODefault.java (+0 -73)
➖ src/main/java/com/mageddo/dnsproxyserver/docker/DockerDAOMock.java (+0 -43)
➖ src/main/java/com/mageddo/dnsproxyserver/docker/DockerNetworkService.java (+0 -77)
➖ src/main/java/com/mageddo/dnsproxyserver/docker/DpsContainerManager.java (+0 -170)
➖ src/main/java/com/mageddo/dnsproxyserver/docker/MatchingContainerService.java (+0 -26)
➖ src/main/java/com/mageddo/dnsproxyserver/docker/NetworkComparator.java (+0 -25)
📝 src/main/java/com/mageddo/dnsproxyserver/docker/application/Containers.java (+5 -2)
📝 src/main/java/com/mageddo/dnsproxyserver/docker/application/DockerConnectionCheck.java (+1 -1)

...and 66 more files

📄 Description

Fixes #444

🔄 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/445 **Author:** [@mageddo](https://github.com/mageddo) **Created:** 5/13/2024 **Status:** ✅ Merged **Merged:** 5/13/2024 **Merged by:** [@mageddo](https://github.com/mageddo) **Base:** `master` ← **Head:** `feat/444` --- ### 📝 Commits (10+) - [`16c7bae`](https://github.com/mageddo/dns-proxy-server/commit/16c7bae4279947f76539454ebc9fc1fd359c1268) refactoring unnecessary parameter - [`f7adfe8`](https://github.com/mageddo/dns-proxy-server/commit/f7adfe83b0be626909e60d99b43e2b3e7bbf8889) refactoring class packages - [`284a538`](https://github.com/mageddo/dns-proxy-server/commit/284a538b65cc8fcd2086d7321276fbfc541d0ce2) isolating container solving in app layer - [`c4d9023`](https://github.com/mageddo/dns-proxy-server/commit/c4d9023c0783ee75dc8493079741bc225ac6271c) a lot of refactoring trying to separate solver.docker module application from dataprovider impl - [`37c37ca`](https://github.com/mageddo/dns-proxy-server/commit/37c37cacf0c9c4c2d520f8550e35a5e7e989e9e7) app compiling errors fixed - [`0d4ba7f`](https://github.com/mageddo/dns-proxy-server/commit/0d4ba7f3b6ae70561bfc0c0a46ccd4774bb3a5eb) testing container mapper - [`d10ca10`](https://github.com/mageddo/dns-proxy-server/commit/d10ca10030c0a46eafa24bac849d6d9dc3664f16) mustSolveSpecifiedNetworkFirst fixed - [`00f8180`](https://github.com/mageddo/dns-proxy-server/commit/00f8180fbcc6224045dcae409230b048dc308899) refactoring test location - [`3d4f23f`](https://github.com/mageddo/dns-proxy-server/commit/3d4f23f775e79c9c60c41119bf14527ce3de081b) new test - [`a41c588`](https://github.com/mageddo/dns-proxy-server/commit/a41c588f21bf1c36a5474fc03f901614807f49a0) creating new test ### 📊 Changes **86 files changed** (+2027 additions, -994 deletions) <details> <summary>View changed files</summary> 📝 `src/main/java/com/mageddo/dnsproxyserver/config/Config.java` (+1 -1) 📝 `src/main/java/com/mageddo/dnsproxyserver/config/ConfigDAO.java` (+2 -0) 📝 `src/main/java/com/mageddo/dnsproxyserver/config/ConfigDAOJson.java` (+5 -0) 📝 `src/main/java/com/mageddo/dnsproxyserver/config/Configs.java` (+1 -1) 📝 `src/main/java/com/mageddo/dnsproxyserver/di/Context.java` (+4 -7) 📝 `src/main/java/com/mageddo/dnsproxyserver/di/module/ModuleDao.java` (+29 -8) 📝 `src/main/java/com/mageddo/dnsproxyserver/di/module/ModuleMap.java` (+12 -12) 📝 `src/main/java/com/mageddo/dnsproxyserver/di/module/ModuleStartup.java` (+1 -1) 📝 `src/main/java/com/mageddo/dnsproxyserver/dnsconfigurator/DnsConfigurators.java` (+3 -2) ➖ `src/main/java/com/mageddo/dnsproxyserver/dnsconfigurator/DpsIpDiscover.java` (+0 -32) ➖ `src/main/java/com/mageddo/dnsproxyserver/docker/ContainerDAO.java` (+0 -12) ➖ `src/main/java/com/mageddo/dnsproxyserver/docker/ContainerSolvingService.java` (+0 -180) ➖ `src/main/java/com/mageddo/dnsproxyserver/docker/DockerDAODefault.java` (+0 -73) ➖ `src/main/java/com/mageddo/dnsproxyserver/docker/DockerDAOMock.java` (+0 -43) ➖ `src/main/java/com/mageddo/dnsproxyserver/docker/DockerNetworkService.java` (+0 -77) ➖ `src/main/java/com/mageddo/dnsproxyserver/docker/DpsContainerManager.java` (+0 -170) ➖ `src/main/java/com/mageddo/dnsproxyserver/docker/MatchingContainerService.java` (+0 -26) ➖ `src/main/java/com/mageddo/dnsproxyserver/docker/NetworkComparator.java` (+0 -25) 📝 `src/main/java/com/mageddo/dnsproxyserver/docker/application/Containers.java` (+5 -2) 📝 `src/main/java/com/mageddo/dnsproxyserver/docker/application/DockerConnectionCheck.java` (+1 -1) _...and 66 more files_ </details> ### 📄 Description Fixes #444 --- <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:26 +03:00
Sign in to join this conversation.
No Branch/Tag specified
Branches Tags
master
gh-pages
develop
feat/138_2
feat/629
feat/594
5.9.0
5.9.0-snapshot
5.8.4
5.8.4-snapshot
5.8.3-snapshot
5.8.2-snapshot
5.8.1-snapshot
5.8.0-snapshot
5.7.0-snapshot
5.6.2-snapshot
5.6.1-snapshot
5.6.0-snapshot
5.5.1-snapshot
5.5.0-snapshot
5.4.0-snapshot
5.3.0-snapshot
5.2.0-snapshot
5.1.3-snapshot
5.1.2-snapshot
5.1.0-snapshot
5.0.5-snapshot
5.0.3-snapshot
5.0.2-snapshot
5.0.1-snapshot
5.0.0-snapshot
4.3.0-snapshot
4.2.0-snapshot
4.0.0
4.0.0-snapshot
3.32.7
3.32.7-snapshot
3.32.6
3.32.6-snapshot
3.32.5-snapshot
3.32.4
3.32.4-snapshot
3.32.3
3.32.3-snapshot
3.32.2-snapshot
3.32.1-snapshot
3.32.0-snapshot
3.31.1-snapshot
3.31.0-snapshot
3.30.5-snapshot
3.30.4
3.30.4-snapshot
3.30.3-snapshot
3.30.2-snapshot
3.30.1
3.30.1-snapshot
3.30.0-snapshot
3.29.0-snapshot
3.27.0
3.27.0-snapshot
3.26.0-snapshot
3.25.14
3.25.14-snapshot
3.25.13-snapshot
3.25.12-snapshot
3.25.11
3.25.11-snapshot
3.25.10
3.25.10-snapshot
3.25.9-snapshot
3.25.8-snapshot
3.25.7-snapshot
3.25.6-snapshot
3.25.4-snapshot
3.25.3-snapshot
3.25.2-snapshot
3.25.1-snapshot
3.25.0-snapshot
3.24.2
3.24.2-snapshot
3.24.1
3.24.1-snapshot
3.24.0-snapshot
3.23.0-snapshot
3.22.2-snapshot
3.22.3-snapshot
3.22.1-snapshot
3.21.3-snapshot
3.22.0-snapshot
3.21.2-snapshot
3.21.1-snapshot
3.20.0-snapshot
3.21.0-snapshot
3.19.7-snapshot
3.19.6-snapshot
3.19.5-snapshot
3.19.4-snapshot
3.19.3-snapshot
3.19.2-snapshot
3.19.1
3.19.1-snapshot
3.19.0
3.18.5-snapshot
3.18.4-snapshot
3.18.3-snapshot
3.18.2
3.18.2-snapshot
3.18.1-snapshot
3.18.0-snapshot
3.17.3-snapshot
3.17.2-snapshot
3.17.1-snapshot
3.17.0-snapshot
3.16.3-snapshot
3.16.2-snapshot
3.16.1-snapshot
3.16.0-snapshot
3.15.13-snapshot
3.15.12-snapshot
3.15.11-snapshot
3.15.10-snapshot
3.15.9-snapshot
3.15.8-snapshot
3.15.7-snapshot
3.15.6-snapshot
3.15.5-snapshot
3.15.4-snapshot
3.15.3-snapshot
3.15.2-snapshot
3.15.1-snapshot
3.15.0-snapshot
3.14.5
3.14.4
3.14.3
3.14.3-snapshot
3.14.2-snapshot
3.14.1-snapshot
3.14.0-snapshot
3.13.1
3.13.1-snapshot
3.13.0-snapshot
3.12.1
3.12.0
3.11.0
3.10.2-snapshot
3.10.3-snapshot
3.10.4-snapshot
3.10.5-snapshot
3.10.1-snapshot
3.9.2
3.9.1
3.9.0
3.8.1-last-version-with-quarkus
3.8.1
3.8.0
3.7.0
3.6.0
3.5.3
3.5.2
3.5.0
3.4.0-beta
3.3.0-beta
3.2.5-beta
3.2.4-beta
3.2.3-beta
3.2.2-beta
3.2.1-beta
3.2.0-beta
3.1.7-beta
3.1.6-beta
3.1.5-beta
3.1.4-beta
3.1.3-beta
3.1.2-beta
b-jre-8
3.1.1-beta
3.1.0-beta
3.0.4-beta
3.0.3-beta
3.0.2-beta
3.0.1-beta
v2-golang
2.19.5
2.19.4
2.19.3
2.19.2
2.19.1
2.19.0
2.18.7
2.18.6
2.18.5
2.18.4
2.18.3
2.18.2
2.18.1
2.18.0
2.17.4
2.17.3
2.17.2
2.17.1
2.17.0
2.16.0
2.15.0
2.14.6
2.14.5
2.14.4
2.14.2
2.14.1
2.14.0
2.13.2
2.13.1
2.13.0
2.12.0
2.11.0
2.10.4
2.10.3
2.10.2
2.10.1
2.10.0
2.9.1
2.9.0
2.8.0
2.7.0
2.6.1
2.6.0
2.5.4
2.5.3
2.5.2
2.5.1
2.5.0
2.4.1
2.4.0
2.3.3
2.2.3
2.2.2
2.2.1
2.2.0
2.1.8
2.1.7
2.1.6
2.1.5
2.1.2
2.1.1
2.1.0
2.0.21
2.0.20
2.0.19
2.0.18
2.0.17
2.0.16
2.0.9
2.0.5
2.0.4
v1-nodejs
1.8.1
1.8.0
1.7.7
1.7.6
1.7.5
1.7.4
1.7.3
1.7.2
1.7.1
1.7.0
1.6.9
1.6.8
1.6.7
1.6.6
1.6.5
1.6.4
1.6.3
1.6.2
1.6.1
1.6.0
1.5.0
1.4.0
1.3.5
1.3.4
1.3.3
1.3.2
1.3.1
1.3.0
peteris-version
1.1.4-peteris-version
1.2.0
1.1.3
1.1.2
1.1.1
1.1.0
1.0.1
1.0.0
Labels
Clear labels   
bug

   
confirmed

   
discussion

   
duplicate

   
enhancement

   
feature

   
feature-request

   
not-planned

   
pull-request

Mirrored from GitHub Pull Request

  
secondary-feature

   
stale

   
triage

   
waiting-feedback
No labels
bug
confirmed
discussion
duplicate
enhancement
feature
feature-request
not-planned
pull-request
secondary-feature
stale
triage
waiting-feedback
Milestone
Clear milestone
No items
No milestone
Projects
Clear projects
No items
No project
Assignees
Clear assignees
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#498
No description provided.