mirror of
https://github.com/mageddo/dns-proxy-server.git
synced 2026-04-25 09:25:56 +03:00
[PR #304] [MERGED] DPS 3 - Bind UDP server to all available interfaces #408
Labels
No labels
bug
confirmed
discussion
duplicate
enhancement
feature
feature-request
not-planned
pull-request
secondary-feature
stale
triage
waiting-feedback
No milestone
No project
No assignees
1 participant
Notifications
Due date
No due date set.
Dependencies
No dependencies set.
Reference
starred/dns-proxy-server-mageddo#408
Loading…
Add table
Add a link
Reference in a new issue
No description provided.
Delete branch "%!s()"
Deleting a branch is permanent. Although the deleted branch may continue to exist for a short time before it actually gets removed, it CANNOT be undone in most cases. Continue?
📋 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:
master← Head:all-interfaces-bind-fix📝 Commits (1)
9aae256Bind 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.