[PR #296] [MERGED] DPS 3 - Minor fixes #400

New issue

Closed

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

kerem commented

2026-02-26 04:35:07 +03:00

Owner

📋 Pull Request Information

Original PR: https://github.com/mageddo/dns-proxy-server/pull/296
Author: @mageddo
Created: 2/22/2023
Status: ✅ Merged
Merged: 2/22/2023
Merged by: @mageddo

Base: master ← Head: dps-3-minor-fixes

📝 Commits (7)

42d4790 disabling quarkus banner
b264fba fixing was caching dns server timeout
8e08e5f Also try to solve conf from dps binary path
9c6698e Fixing log level convention
5095cde release notes
dbad175 remove test which dont makes sense
b0d8ed4 testing already correct case

📊 Changes

15 files changed (+222 additions, -16 deletions)

View changed files

📝 RELEASE-NOTES.md (+8 -0)
📝 gradle.properties (+1 -1)
📝 src/main/java/com/mageddo/dnsproxyserver/config/Configs.java (+8 -1)
📝 src/main/java/com/mageddo/dnsproxyserver/config/entrypoint/LogLevel.java (+22 -3)
📝 src/main/java/com/mageddo/dnsproxyserver/quarkus/QuarkusConfig.java (+3 -1)
📝 src/main/java/com/mageddo/dnsproxyserver/server/dns/solver/SolverRemote.java (+4 -1)
📝 src/main/java/com/mageddo/dnsproxyserver/server/dns/solver/SolversCache.java (+8 -8)
📝 src/main/java/com/mageddo/utils/Files.java (+4 -0)
➕ src/main/java/com/mageddo/utils/Runtime.java (+72 -0)
📝 src/main/resources/application.properties (+2 -0)
📝 src/test/java/com/mageddo/dnsproxyserver/config/ConfigsTest.java (+28 -0)
➕ src/test/java/com/mageddo/dnsproxyserver/quarkus/QuarkusConfigTest.java (+23 -0)
📝 src/test/java/com/mageddo/dnsproxyserver/server/dns/solver/SolversCacheTest.java (+14 -0)
➕ src/test/java/com/mageddo/dnsproxyserver/templates/ConfigFlagTemplates.java (+15 -0)
📝 src/test/java/com/mageddo/dnsproxyserver/templates/ConfigTemplates.java (+10 -1)

📄 Description

Timed out remote servers responses are being cached, it cant happen, when its a timeout cant cache but when the record really dont exists, like AAAA for bookmarks.mageddo.com must cache.
Must read conf from dps binary path not the current os path where dps was called
Log level not being respected, at least no when running in native image binary
Remove quarkus splash logo

Partially fixing https://github.com/mageddo/dns-proxy-server/issues/267

_{🔄 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/296 **Author:** [@mageddo](https://github.com/mageddo) **Created:** 2/22/2023 **Status:** ✅ Merged **Merged:** 2/22/2023 **Merged by:** [@mageddo](https://github.com/mageddo) **Base:** `master` ← **Head:** `dps-3-minor-fixes` --- ### 📝 Commits (7) - [`42d4790`](https://github.com/mageddo/dns-proxy-server/commit/42d4790e399de9ed50e801e28b02453a79335bd8) disabling quarkus banner - [`b264fba`](https://github.com/mageddo/dns-proxy-server/commit/b264fbac7716660d89a7d1b190135d20102932ce) fixing was caching dns server timeout - [`8e08e5f`](https://github.com/mageddo/dns-proxy-server/commit/8e08e5f9798a2bf0f4b33fe56c1759d8164dda44) Also try to solve conf from dps binary path - [`9c6698e`](https://github.com/mageddo/dns-proxy-server/commit/9c6698e33c15e5f61652850d559d9ea065fb69bf) Fixing log level convention - [`5095cde`](https://github.com/mageddo/dns-proxy-server/commit/5095cde4945721ae3717c2b4f5f34a8c1cb8df60) release notes - [`dbad175`](https://github.com/mageddo/dns-proxy-server/commit/dbad1759d4b6b5af06d2f727a4ac745d84c21447) remove test which dont makes sense - [`b0d8ed4`](https://github.com/mageddo/dns-proxy-server/commit/b0d8ed498a2e0ae902586b19a5815979c2ed5e73) testing already correct case ### 📊 Changes **15 files changed** (+222 additions, -16 deletions) <details> <summary>View changed files</summary> 📝 `RELEASE-NOTES.md` (+8 -0) 📝 `gradle.properties` (+1 -1) 📝 `src/main/java/com/mageddo/dnsproxyserver/config/Configs.java` (+8 -1) 📝 `src/main/java/com/mageddo/dnsproxyserver/config/entrypoint/LogLevel.java` (+22 -3) 📝 `src/main/java/com/mageddo/dnsproxyserver/quarkus/QuarkusConfig.java` (+3 -1) 📝 `src/main/java/com/mageddo/dnsproxyserver/server/dns/solver/SolverRemote.java` (+4 -1) 📝 `src/main/java/com/mageddo/dnsproxyserver/server/dns/solver/SolversCache.java` (+8 -8) 📝 `src/main/java/com/mageddo/utils/Files.java` (+4 -0) ➕ `src/main/java/com/mageddo/utils/Runtime.java` (+72 -0) 📝 `src/main/resources/application.properties` (+2 -0) 📝 `src/test/java/com/mageddo/dnsproxyserver/config/ConfigsTest.java` (+28 -0) ➕ `src/test/java/com/mageddo/dnsproxyserver/quarkus/QuarkusConfigTest.java` (+23 -0) 📝 `src/test/java/com/mageddo/dnsproxyserver/server/dns/solver/SolversCacheTest.java` (+14 -0) ➕ `src/test/java/com/mageddo/dnsproxyserver/templates/ConfigFlagTemplates.java` (+15 -0) 📝 `src/test/java/com/mageddo/dnsproxyserver/templates/ConfigTemplates.java` (+10 -1) </details> ### 📄 Description * [x] Timed out remote servers responses are being cached, it cant happen, when its a timeout cant cache but when the record really dont exists, like AAAA for bookmarks.mageddo.com must cache. * [x] Must read conf from dps binary path not the current os path where dps was called * [x] Log level not being respected, at least no when running in native image binary * [x] Remove quarkus splash logo Partially fixing https://github.com/mageddo/dns-proxy-server/issues/267 --- <sub>🔄 This issue represents a GitHub Pull Request. It cannot be merged through Gitea due to API limitations.</sub>