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 #589] [MERGED] Solver Remote Refactoring #605

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

📋 Pull Request Information

Original PR: https://github.com/mageddo/dns-proxy-server/pull/589
Author: @mageddo
Created: 10/21/2024
Status: ✅ Merged
Merged: 10/21/2024
Merged by: @mageddo

Base: master ← Head: feat/533-solver-remote-refactoring

📝 Commits (9)

📊 Changes

21 files changed (+387 additions, -309 deletions)

View changed files

📝 RELEASE-NOTES.md (+3 -0)
📝 gradle.properties (+1 -1)
📝 src/main/java/com/mageddo/commons/circuitbreaker/CircuitCheckException.java (+4 -0)
📝 src/main/java/com/mageddo/dnsproxyserver/solver/SolverRemote.java (+5 -88)
📝 src/main/java/com/mageddo/dnsproxyserver/solver/remote/Request.java (+4 -0)
📝 src/main/java/com/mageddo/dnsproxyserver/solver/remote/application/CircuitBreakerService.java (+6 -6)
➕ src/main/java/com/mageddo/dnsproxyserver/solver/remote/application/RemoteResultSupplier.java (+63 -0)
➕ src/main/java/com/mageddo/dnsproxyserver/solver/remote/application/ResultSupplier.java (+10 -0)
📝 src/main/java/com/mageddo/dnsproxyserver/solver/remote/application/failsafe/CircuitBreakerFactory.java (+14 -9)
📝 src/main/java/com/mageddo/dnsproxyserver/solver/remote/application/failsafe/CircuitBreakerPingCheckerService.java (+2 -2)
➕ src/main/java/com/mageddo/dnsproxyserver/solver/remote/application/mapper/ResultMapper.java (+87 -0)
➖ src/main/java/com/mageddo/dnsproxyserver/solver/remote/circuitbreaker/application/CircuitBreakerExecutorService.java (+0 -33)
📝 src/main/java/com/mageddo/dnsproxyserver/solver/remote/mapper/ResolverMapper.java (+5 -2)
📝 src/main/java/com/mageddo/net/IpAddr.java (+2 -0)
📝 src/test/java/com/mageddo/dnsproxyserver/solver/SolverRemoteTest.java (+0 -161)
➕ src/test/java/com/mageddo/dnsproxyserver/solver/remote/application/RemoteResultSupplierTest.java (+68 -0)
📝 src/test/java/com/mageddo/dnsproxyserver/solver/remote/application/failsafe/CircuitBreakerFactoryTest.java (+6 -6)
➕ src/test/java/com/mageddo/dnsproxyserver/solver/remote/application/mapper/ResultMapperTest.java (+88 -0)
📝 src/test/java/testing/templates/InetSocketAddressTemplates.java (+9 -0)
📝 src/test/java/testing/templates/MessageTemplates.java (+7 -0)

...and 1 more files

📄 Description

Relates to #533

To make the remote dns server address clear at the logs when getting timeout, I've refactor SolverRemote into two more classes separating the responsabilities

🔄 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/589 **Author:** [@mageddo](https://github.com/mageddo) **Created:** 10/21/2024 **Status:** ✅ Merged **Merged:** 10/21/2024 **Merged by:** [@mageddo](https://github.com/mageddo) **Base:** `master` ← **Head:** `feat/533-solver-remote-refactoring` --- ### 📝 Commits (9) - [`123e868`](https://github.com/mageddo/dns-proxy-server/commit/123e868025ba5c167e0bd9233ca4f94f01d6461d) refactoring solver remote class - [`cd3aac0`](https://github.com/mageddo/dns-proxy-server/commit/cd3aac0b8d1c8bb3c825e9d0d32402fc7afbd060) fixing compiling errors - [`f3b922e`](https://github.com/mageddo/dns-proxy-server/commit/f3b922e66bc82d67b50b9807608e56ac2ce0b4c5) fixing compiling errors - [`5b55859`](https://github.com/mageddo/dns-proxy-server/commit/5b5585930d8c2b46b1556840b9a4faa414d21b7b) reverting - [`95fe342`](https://github.com/mageddo/dns-proxy-server/commit/95fe342053577a5f248d0d70fdd6f85200913d44) fixing compiling errors - [`fb7b25d`](https://github.com/mageddo/dns-proxy-server/commit/fb7b25dc95dd89d39044e3ca5a81005ad49464dd) fixing compiling errors - [`eeff8d9`](https://github.com/mageddo/dns-proxy-server/commit/eeff8d915b325f2ad84283f044d5cce7e867cbfd) release notes - [`1fd462f`](https://github.com/mageddo/dns-proxy-server/commit/1fd462f6d5e82611560519ea494b9e23a25f9fdc) [Gradle Release Plugin] - new version commit: '3.30.3-snapshot'. - [`467a4e7`](https://github.com/mageddo/dns-proxy-server/commit/467a4e799d4d345d86fc8227194d30adaf3daec9) refactoring ### 📊 Changes **21 files changed** (+387 additions, -309 deletions) <details> <summary>View changed files</summary> 📝 `RELEASE-NOTES.md` (+3 -0) 📝 `gradle.properties` (+1 -1) 📝 `src/main/java/com/mageddo/commons/circuitbreaker/CircuitCheckException.java` (+4 -0) 📝 `src/main/java/com/mageddo/dnsproxyserver/solver/SolverRemote.java` (+5 -88) 📝 `src/main/java/com/mageddo/dnsproxyserver/solver/remote/Request.java` (+4 -0) 📝 `src/main/java/com/mageddo/dnsproxyserver/solver/remote/application/CircuitBreakerService.java` (+6 -6) ➕ `src/main/java/com/mageddo/dnsproxyserver/solver/remote/application/RemoteResultSupplier.java` (+63 -0) ➕ `src/main/java/com/mageddo/dnsproxyserver/solver/remote/application/ResultSupplier.java` (+10 -0) 📝 `src/main/java/com/mageddo/dnsproxyserver/solver/remote/application/failsafe/CircuitBreakerFactory.java` (+14 -9) 📝 `src/main/java/com/mageddo/dnsproxyserver/solver/remote/application/failsafe/CircuitBreakerPingCheckerService.java` (+2 -2) ➕ `src/main/java/com/mageddo/dnsproxyserver/solver/remote/application/mapper/ResultMapper.java` (+87 -0) ➖ `src/main/java/com/mageddo/dnsproxyserver/solver/remote/circuitbreaker/application/CircuitBreakerExecutorService.java` (+0 -33) 📝 `src/main/java/com/mageddo/dnsproxyserver/solver/remote/mapper/ResolverMapper.java` (+5 -2) 📝 `src/main/java/com/mageddo/net/IpAddr.java` (+2 -0) 📝 `src/test/java/com/mageddo/dnsproxyserver/solver/SolverRemoteTest.java` (+0 -161) ➕ `src/test/java/com/mageddo/dnsproxyserver/solver/remote/application/RemoteResultSupplierTest.java` (+68 -0) 📝 `src/test/java/com/mageddo/dnsproxyserver/solver/remote/application/failsafe/CircuitBreakerFactoryTest.java` (+6 -6) ➕ `src/test/java/com/mageddo/dnsproxyserver/solver/remote/application/mapper/ResultMapperTest.java` (+88 -0) 📝 `src/test/java/testing/templates/InetSocketAddressTemplates.java` (+9 -0) 📝 `src/test/java/testing/templates/MessageTemplates.java` (+7 -0) _...and 1 more files_ </details> ### 📄 Description Relates to #533 To make the remote dns server address clear at the logs when getting timeout, I've refactor SolverRemote into two more classes separating the responsabilities --- <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:45 +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#605
No description provided.