[PR #635] [MERGED] Config v3 - EnvConverter #594 #634

New issue

Closed

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

kerem commented

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

Owner

📋 Pull Request Information

Original PR: https://github.com/mageddo/dns-proxy-server/pull/635
Author: @mageddo
Created: 9/22/2025
Status: ✅ Merged
Merged: 9/22/2025
Merged by: @mageddo

Base: master ← Head: codex/implement-envmapper-tojson-method

📝 Commits (4)

1f2703d feat: convert environment variables to config v3
5557052 adjusts
f600da9 adjusts
c38fce7 adjusts

📊 Changes

8 files changed (+316 additions, -3 deletions)

View changed files

📝 build.gradle (+1 -0)
➕ src/main/java/com/mageddo/dataformat/env/EnvMapper.java (+192 -0)
📝 src/main/java/com/mageddo/dnsproxyserver/config/provider/dataformatv3/converter/EnvConverter.java (+21 -2)
📝 src/main/java/com/mageddo/dnsproxyserver/utils/Numbers.java (+4 -0)
➕ src/test/java/com/mageddo/dataformat/env/EnvMapperTest.java (+27 -0)
➕ src/test/java/com/mageddo/dnsproxyserver/config/provider/dataformatv3/converter/EnvConverterTest.java (+26 -0)
➕ src/test/java/com/mageddo/dnsproxyserver/config/provider/dataformatv3/templates/ConfigV3EnvTemplates.java (+44 -0)
📝 src/test/java/com/mageddo/dnsproxyserver/config/provider/dataformatv3/templates/ConfigV3Templates.java (+1 -1)

📄 Description

Summary

add an EnvMapper that turns prefixed environment variables into JSON structures
use the new mapper in EnvConverter to build ConfigV3 from DPS_ variables
cover the mapper and converter with unit tests alongside reusable environment templates

Testing

./gradlew test (fails: CircuitBreakerDelegateSelfObservableTest.mustHalfOpenCircuitAfterConfiguredTimeAndSatisfyHealthCheck due to UnnecessaryStubbingException)

https://chatgpt.com/codex/tasks/task_e_68d16fe7e920832abc1d41825bceff7c

_{🔄 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/635 **Author:** [@mageddo](https://github.com/mageddo) **Created:** 9/22/2025 **Status:** ✅ Merged **Merged:** 9/22/2025 **Merged by:** [@mageddo](https://github.com/mageddo) **Base:** `master` ← **Head:** `codex/implement-envmapper-tojson-method` --- ### 📝 Commits (4) - [`1f2703d`](https://github.com/mageddo/dns-proxy-server/commit/1f2703d633707204b14d69571eafb99be869c20d) feat: convert environment variables to config v3 - [`5557052`](https://github.com/mageddo/dns-proxy-server/commit/5557052d33931c81b2dd08ade5d80a405be55064) adjusts - [`f600da9`](https://github.com/mageddo/dns-proxy-server/commit/f600da9eb173b623e9fd568928c04d05ade00cc2) adjusts - [`c38fce7`](https://github.com/mageddo/dns-proxy-server/commit/c38fce704894d531844f1da2662c7cb61e377586) adjusts ### 📊 Changes **8 files changed** (+316 additions, -3 deletions) <details> <summary>View changed files</summary> 📝 `build.gradle` (+1 -0) ➕ `src/main/java/com/mageddo/dataformat/env/EnvMapper.java` (+192 -0) 📝 `src/main/java/com/mageddo/dnsproxyserver/config/provider/dataformatv3/converter/EnvConverter.java` (+21 -2) 📝 `src/main/java/com/mageddo/dnsproxyserver/utils/Numbers.java` (+4 -0) ➕ `src/test/java/com/mageddo/dataformat/env/EnvMapperTest.java` (+27 -0) ➕ `src/test/java/com/mageddo/dnsproxyserver/config/provider/dataformatv3/converter/EnvConverterTest.java` (+26 -0) ➕ `src/test/java/com/mageddo/dnsproxyserver/config/provider/dataformatv3/templates/ConfigV3EnvTemplates.java` (+44 -0) 📝 `src/test/java/com/mageddo/dnsproxyserver/config/provider/dataformatv3/templates/ConfigV3Templates.java` (+1 -1) </details> ### 📄 Description ## Summary - add an EnvMapper that turns prefixed environment variables into JSON structures - use the new mapper in EnvConverter to build ConfigV3 from DPS_ variables - cover the mapper and converter with unit tests alongside reusable environment templates ## Testing - `./gradlew test` *(fails: CircuitBreakerDelegateSelfObservableTest.mustHalfOpenCircuitAfterConfiguredTimeAndSatisfyHealthCheck due to UnnecessaryStubbingException)* ------ https://chatgpt.com/codex/tasks/task_e_68d16fe7e920832abc1d41825bceff7c --- <sub>🔄 This issue represents a GitHub Pull Request. It cannot be merged through Gitea due to API limitations.</sub>