[PR #460] [MERGED] Refactoring SolveRemote Module #509

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

📋 Pull Request Information

Original PR: https://github.com/mageddo/dns-proxy-server/pull/460
Author: @mageddo
Created: 6/3/2024
Status: Merged
Merged: 6/3/2024
Merged by: @mageddo

Base: masterHead: feat/455


📝 Commits (10+)

  • fd4656e creating and binding service to clear the cache
  • 12ff231 refactoring
  • ea36e8f refactoriong
  • c5580b7 refactoring and fixed test
  • 29f0f03 refactoring
  • ada0bc7 reactivated the tests
  • 3eb548c refactoring and testing circuit breaker
  • aefdadf unnecessary test, the current active test shows that when a circuit exception or CircuitBreakerOpenException is thrown, it will return an empty result, not throws exception
  • 9a22281 refactoring
  • 3d21503 release notes

📊 Changes

20 files changed (+404 additions, -239 deletions)

View changed files

📝 RELEASE-NOTES.md (+3 -0)
📝 gradle.properties (+1 -1)
📝 src/main/java/com/mageddo/dnsproxyserver/di/Context.java (+3 -1)
📝 src/main/java/com/mageddo/dnsproxyserver/quarkus/QuarkusConfig.java (+3 -9)
📝 src/main/java/com/mageddo/dnsproxyserver/solver/SolverCacheFactory.java (+7 -3)
📝 src/main/java/com/mageddo/dnsproxyserver/solver/SolverRemote.java (+5 -125)
src/main/java/com/mageddo/dnsproxyserver/solver/remote/CircuitBreakerService.java (+9 -0)
src/main/java/com/mageddo/dnsproxyserver/solver/remote/Request.java (+54 -0)
src/main/java/com/mageddo/dnsproxyserver/solver/remote/Result.java (+47 -0)
src/main/java/com/mageddo/dnsproxyserver/solver/remote/application/CircuitBreakerFactory.java (+62 -0)
src/main/java/com/mageddo/dnsproxyserver/solver/remote/application/CircuitBreakerFailSafeService.java (+44 -0)
src/main/java/com/mageddo/dnsproxyserver/solver/remote/application/CircuitBreakerNonResilientService.java (+14 -0)
src/main/java/com/mageddo/dnsproxyserver/solver/remote/configurator/SolverRemoteModule.java (+21 -0)
src/main/java/com/mageddo/dnsproxyserver/solver/remote/dataprovider/SolverConsistencyGuaranteeDAO.java (+5 -0)
src/main/java/com/mageddo/dnsproxyserver/solver/remote/dataprovider/SolverConsistencyGuaranteeDAOImpl.java (+21 -0)
src/main/java/com/mageddo/dnsproxyserver/solver/remote/mapper/CircuitBreakerStateMapper.java (+9 -0)
src/main/java/com/mageddo/dnsproxyserver/solver/remote/mapper/ResolverMapper.java (+24 -0)
📝 src/test/java/com/mageddo/dnsproxyserver/solver/SolverRemoteTest.java (+4 -100)
src/test/java/com/mageddo/dnsproxyserver/solver/remote/application/CircuitBreakerFailSafeServiceCompTest.java (+49 -0)
src/test/java/testing/templates/solver/remote/RequestTemplates.java (+19 -0)

📄 Description

See #459


🔄 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/460 **Author:** [@mageddo](https://github.com/mageddo) **Created:** 6/3/2024 **Status:** ✅ Merged **Merged:** 6/3/2024 **Merged by:** [@mageddo](https://github.com/mageddo) **Base:** `master` ← **Head:** `feat/455` --- ### 📝 Commits (10+) - [`fd4656e`](https://github.com/mageddo/dns-proxy-server/commit/fd4656efb4ea116146b95831625dd4aa048dd2d3) creating and binding service to clear the cache - [`12ff231`](https://github.com/mageddo/dns-proxy-server/commit/12ff2315ab2011e36d66dfe653f902ae365d2f51) refactoring - [`ea36e8f`](https://github.com/mageddo/dns-proxy-server/commit/ea36e8fab6b83733c5e097bfc79c02290d735ed4) refactoriong - [`c5580b7`](https://github.com/mageddo/dns-proxy-server/commit/c5580b741c958408ce8b225a68a4eb27e46cdaa9) refactoring and fixed test - [`29f0f03`](https://github.com/mageddo/dns-proxy-server/commit/29f0f03dd794f6320bd51b9bdd39c7242409de4a) refactoring - [`ada0bc7`](https://github.com/mageddo/dns-proxy-server/commit/ada0bc7f023ed933134b46677ef0c3691a4f90df) reactivated the tests - [`3eb548c`](https://github.com/mageddo/dns-proxy-server/commit/3eb548cde524297bd4a58ab3478561f806c9cd1a) refactoring and testing circuit breaker - [`aefdadf`](https://github.com/mageddo/dns-proxy-server/commit/aefdadfc82e7d4fa88c552d5d49f2c87bb741d04) unnecessary test, the current active test shows that when a circuit exception or CircuitBreakerOpenException is thrown, it will return an empty result, not throws exception - [`9a22281`](https://github.com/mageddo/dns-proxy-server/commit/9a22281839c30cd3b61da74a7f94fbdabec2154b) refactoring - [`3d21503`](https://github.com/mageddo/dns-proxy-server/commit/3d21503791f5c9404f4e947a1a8575aa76cadcd9) release notes ### 📊 Changes **20 files changed** (+404 additions, -239 deletions) <details> <summary>View changed files</summary> 📝 `RELEASE-NOTES.md` (+3 -0) 📝 `gradle.properties` (+1 -1) 📝 `src/main/java/com/mageddo/dnsproxyserver/di/Context.java` (+3 -1) 📝 `src/main/java/com/mageddo/dnsproxyserver/quarkus/QuarkusConfig.java` (+3 -9) 📝 `src/main/java/com/mageddo/dnsproxyserver/solver/SolverCacheFactory.java` (+7 -3) 📝 `src/main/java/com/mageddo/dnsproxyserver/solver/SolverRemote.java` (+5 -125) ➕ `src/main/java/com/mageddo/dnsproxyserver/solver/remote/CircuitBreakerService.java` (+9 -0) ➕ `src/main/java/com/mageddo/dnsproxyserver/solver/remote/Request.java` (+54 -0) ➕ `src/main/java/com/mageddo/dnsproxyserver/solver/remote/Result.java` (+47 -0) ➕ `src/main/java/com/mageddo/dnsproxyserver/solver/remote/application/CircuitBreakerFactory.java` (+62 -0) ➕ `src/main/java/com/mageddo/dnsproxyserver/solver/remote/application/CircuitBreakerFailSafeService.java` (+44 -0) ➕ `src/main/java/com/mageddo/dnsproxyserver/solver/remote/application/CircuitBreakerNonResilientService.java` (+14 -0) ➕ `src/main/java/com/mageddo/dnsproxyserver/solver/remote/configurator/SolverRemoteModule.java` (+21 -0) ➕ `src/main/java/com/mageddo/dnsproxyserver/solver/remote/dataprovider/SolverConsistencyGuaranteeDAO.java` (+5 -0) ➕ `src/main/java/com/mageddo/dnsproxyserver/solver/remote/dataprovider/SolverConsistencyGuaranteeDAOImpl.java` (+21 -0) ➕ `src/main/java/com/mageddo/dnsproxyserver/solver/remote/mapper/CircuitBreakerStateMapper.java` (+9 -0) ➕ `src/main/java/com/mageddo/dnsproxyserver/solver/remote/mapper/ResolverMapper.java` (+24 -0) 📝 `src/test/java/com/mageddo/dnsproxyserver/solver/SolverRemoteTest.java` (+4 -100) ➕ `src/test/java/com/mageddo/dnsproxyserver/solver/remote/application/CircuitBreakerFailSafeServiceCompTest.java` (+49 -0) ➕ `src/test/java/testing/templates/solver/remote/RequestTemplates.java` (+19 -0) </details> ### 📄 Description See #459 --- <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:29 +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#509
No description provided.