[PR #310] [MERGED] DPS 3 - TCP Server Partially message read fix #411

New issue

Closed

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

kerem commented

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

Owner

📋 Pull Request Information

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

Base: master ← Head: feat/308

📝 Commits (10+)

03b24f9 better log desc
30cc4ed refactoring code to build tests
12b1917 making the test
259d4b9 mroe structuring
7bca366 Testing and fixing
a63c599 unnecessary class
44aeccc java doc
426fa6e Using interface instead of impl
9010996 javadoc
bfd68e3 Using interface instead of impl

📊 Changes

16 files changed (+420 additions, -168 deletions)

View changed files

➕ .gitattributes (+28 -0)
📝 RELEASE-NOTES.md (+3 -0)
📝 gradle.properties (+1 -1)
📝 src/main/java/com/mageddo/dnsproxyserver/config/Configs.java (+3 -3)
📝 src/main/java/com/mageddo/dnsproxyserver/config/entrypoint/ConfigFlag.java (+1 -0)
➕ src/main/java/com/mageddo/dnsproxyserver/server/dns/DnsQueryTCPHandler.java (+90 -0)
📝 src/main/java/com/mageddo/dnsproxyserver/server/dns/RequestHandler.java (+8 -88)
➕ src/main/java/com/mageddo/dnsproxyserver/server/dns/RequestHandlerDefault.java (+94 -0)
➕ src/main/java/com/mageddo/dnsproxyserver/server/dns/RequestHandlerMock.java (+29 -0)
📝 src/main/java/com/mageddo/dnsproxyserver/server/dns/SimpleServer.java (+2 -70)
📝 src/main/java/com/mageddo/dnsproxyserver/server/dns/SocketClientMessageHandler.java (+3 -0)
📝 src/main/java/com/mageddo/dnsproxyserver/server/dns/TCPServer.java (+1 -1)
➕ src/main/java/com/mageddo/utils/Shorts.java (+19 -0)
📝 src/test/java/com/mageddo/dnsproxyserver/config/ConfigsTest.java (+6 -5)
➕ src/test/java/com/mageddo/dnsproxyserver/server/dns/DnsQueryTCPHandlerTest.java (+96 -0)
➕ src/test/java/com/mageddo/dnsproxyserver/templates/SocketClientTemplates.java (+36 -0)

📄 Description

Partially fixing #308

_{🔄 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/310 **Author:** [@mageddo](https://github.com/mageddo) **Created:** 2/24/2023 **Status:** ✅ Merged **Merged:** 2/24/2023 **Merged by:** [@mageddo](https://github.com/mageddo) **Base:** `master` ← **Head:** `feat/308` --- ### 📝 Commits (10+) - [`03b24f9`](https://github.com/mageddo/dns-proxy-server/commit/03b24f9b55b45500996c4074b30d1d52053e0157) better log desc - [`30cc4ed`](https://github.com/mageddo/dns-proxy-server/commit/30cc4edce97fe6a7c985c0a67d0646b3c28564d2) refactoring code to build tests - [`12b1917`](https://github.com/mageddo/dns-proxy-server/commit/12b19171baa2d38e678bc655ad6cdf00cb8622f6) making the test - [`259d4b9`](https://github.com/mageddo/dns-proxy-server/commit/259d4b9371135281c6b84e4c014306e65e937bea) mroe structuring - [`7bca366`](https://github.com/mageddo/dns-proxy-server/commit/7bca3661b4910d7dcab56688f21318324b5d93e3) Testing and fixing - [`a63c599`](https://github.com/mageddo/dns-proxy-server/commit/a63c5991856fe0e83e9a1ff50961e5b0f0cf6627) unnecessary class - [`44aeccc`](https://github.com/mageddo/dns-proxy-server/commit/44aeccc4de5ab47fb65789726f9591e5109b8f52) java doc - [`426fa6e`](https://github.com/mageddo/dns-proxy-server/commit/426fa6e1630224a901ba391a11b645183aee2447) Using interface instead of impl - [`9010996`](https://github.com/mageddo/dns-proxy-server/commit/9010996188e48e7537139211b186518be65e3946) javadoc - [`bfd68e3`](https://github.com/mageddo/dns-proxy-server/commit/bfd68e3de1fe1fecc94f7d9e87c140aff41685ea) Using interface instead of impl ### 📊 Changes **16 files changed** (+420 additions, -168 deletions) <details> <summary>View changed files</summary> ➕ `.gitattributes` (+28 -0) 📝 `RELEASE-NOTES.md` (+3 -0) 📝 `gradle.properties` (+1 -1) 📝 `src/main/java/com/mageddo/dnsproxyserver/config/Configs.java` (+3 -3) 📝 `src/main/java/com/mageddo/dnsproxyserver/config/entrypoint/ConfigFlag.java` (+1 -0) ➕ `src/main/java/com/mageddo/dnsproxyserver/server/dns/DnsQueryTCPHandler.java` (+90 -0) 📝 `src/main/java/com/mageddo/dnsproxyserver/server/dns/RequestHandler.java` (+8 -88) ➕ `src/main/java/com/mageddo/dnsproxyserver/server/dns/RequestHandlerDefault.java` (+94 -0) ➕ `src/main/java/com/mageddo/dnsproxyserver/server/dns/RequestHandlerMock.java` (+29 -0) 📝 `src/main/java/com/mageddo/dnsproxyserver/server/dns/SimpleServer.java` (+2 -70) 📝 `src/main/java/com/mageddo/dnsproxyserver/server/dns/SocketClientMessageHandler.java` (+3 -0) 📝 `src/main/java/com/mageddo/dnsproxyserver/server/dns/TCPServer.java` (+1 -1) ➕ `src/main/java/com/mageddo/utils/Shorts.java` (+19 -0) 📝 `src/test/java/com/mageddo/dnsproxyserver/config/ConfigsTest.java` (+6 -5) ➕ `src/test/java/com/mageddo/dnsproxyserver/server/dns/DnsQueryTCPHandlerTest.java` (+96 -0) ➕ `src/test/java/com/mageddo/dnsproxyserver/templates/SocketClientTemplates.java` (+36 -0) </details> ### 📄 Description Partially fixing #308 --- <sub>🔄 This issue represents a GitHub Pull Request. It cannot be merged through Gitea due to API limitations.</sub>