[PR #304] [MERGED] DPS 3 - Bind UDP server to all available interfaces #408

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

📋 Pull Request Information

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

Base: masterHead: all-interfaces-bind-fix


📝 Commits (1)

  • 9aae256 Bind udp to all available interfaces

📊 Changes

7 files changed (+81 additions, -28 deletions)

View changed files

📝 src/main/java/com/mageddo/dnsproxyserver/net/Networks.java (+18 -6)
📝 src/main/java/com/mageddo/dnsproxyserver/quarkus/QuarkusConfig.java (+1 -1)
📝 src/main/java/com/mageddo/dnsproxyserver/server/dns/ServerStarter.java (+1 -1)
📝 src/main/java/com/mageddo/dnsproxyserver/server/dns/SimpleServer.java (+3 -3)
📝 src/main/java/com/mageddo/dnsproxyserver/server/dns/UDPServer.java (+16 -17)
src/main/java/com/mageddo/dnsproxyserver/server/dns/UDPServerPool.java (+36 -0)
📝 src/main/java/com/mageddo/dnsproxyserver/utils/Ips.java (+6 -0)

📄 Description

I was using WILDCARD to create the DatagramSocket new DatagramSocket(port) but was getting issues, by some reason the dns clients like nslookup wasn't able the get the responses when querying using a virtual network interface like docker, when querying using the real interfaces or 127.0.0.1 it worked.

Solution was to start one UDP server for every interface this way all worked


🔄 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/304 **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:** `all-interfaces-bind-fix` --- ### 📝 Commits (1) - [`9aae256`](https://github.com/mageddo/dns-proxy-server/commit/9aae2569b5a60c53e68495ef430eddc02f620487) Bind udp to all available interfaces ### 📊 Changes **7 files changed** (+81 additions, -28 deletions) <details> <summary>View changed files</summary> 📝 `src/main/java/com/mageddo/dnsproxyserver/net/Networks.java` (+18 -6) 📝 `src/main/java/com/mageddo/dnsproxyserver/quarkus/QuarkusConfig.java` (+1 -1) 📝 `src/main/java/com/mageddo/dnsproxyserver/server/dns/ServerStarter.java` (+1 -1) 📝 `src/main/java/com/mageddo/dnsproxyserver/server/dns/SimpleServer.java` (+3 -3) 📝 `src/main/java/com/mageddo/dnsproxyserver/server/dns/UDPServer.java` (+16 -17) ➕ `src/main/java/com/mageddo/dnsproxyserver/server/dns/UDPServerPool.java` (+36 -0) 📝 `src/main/java/com/mageddo/dnsproxyserver/utils/Ips.java` (+6 -0) </details> ### 📄 Description I was using WILDCARD to create the DatagramSocket `new DatagramSocket(port)` but was getting issues, by some reason the dns clients like nslookup wasn't able the get the responses when querying using a virtual network interface like docker, when querying using the real interfaces or 127.0.0.1 it worked. Solution was to start one UDP server for every interface this way all worked --- <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:08 +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#408
No description provided.