[PR #3023] [CLOSED] DNS forwarder #16029

Open
opened 2026-08-05 03:08:49 -04:00 by saavagebueno · 0 comments
Owner

📋 Pull Request Information

Original PR: https://github.com/netbirdio/netbird/pull/3023
Author: @pappz
Created: 12/11/2024
Status: Closed

Base: dns-interceptorHead: dns-fwd


📝 Commits (5)

  • 65ec185 [management] Add structs for new networks concept (#3006)
  • 99f785d [management] Add name attr to network resource (#3011)
  • ea9841d [management] restructure api files (#3013)
  • c02918d Add dns forwarder service
  • 655891d Filter out unmanaged domains

📊 Changes

41 files changed (+1095 additions, -481 deletions)

View changed files

client/internal/dnsfwd/forwarder.go (+115 -0)
client/internal/dnsfwd/manager.go (+106 -0)
📝 client/internal/engine.go (+51 -12)
📝 management/cmd/management.go (+2 -1)
📝 management/proto/management.pb.go (+69 -43)
📝 management/proto/management.proto (+7 -0)
management/server/http/configs/auth.go (+9 -0)
📝 management/server/http/handler.go (+21 -130)
📝 management/server/http/handlers/accounts/accounts_handler.go (+21 -13)
📝 management/server/http/handlers/accounts/accounts_handler_test.go (+6 -6)
📝 management/server/http/handlers/dns/dns_settings_handler.go (+23 -10)
📝 management/server/http/handlers/dns/dns_settings_handler_test.go (+5 -5)
📝 management/server/http/handlers/dns/nameservers_handler.go (+27 -17)
📝 management/server/http/handlers/dns/nameservers_handler_test.go (+7 -7)
📝 management/server/http/handlers/events/events_handler.go (+16 -9)
📝 management/server/http/handlers/events/events_handler_test.go (+5 -5)
📝 management/server/http/handlers/groups/groups_handler.go (+29 -18)
📝 management/server/http/handlers/groups/groups_handler_test.go (+9 -9)
📝 management/server/http/handlers/peers/peers_handler.go (+24 -15)
📝 management/server/http/handlers/peers/peers_handler_test.go (+3 -3)

...and 21 more files

📄 Description

Describe your changes

Checklist

  • Is it a bug fix
  • Is a typo/documentation fix
  • Is a feature enhancement
  • It is a refactor
  • Created tests that fail without the change (if possible)
  • Extended the README / documentation, if necessary

🔄 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/netbirdio/netbird/pull/3023 **Author:** [@pappz](https://github.com/pappz) **Created:** 12/11/2024 **Status:** ❌ Closed **Base:** `dns-interceptor` ← **Head:** `dns-fwd` --- ### 📝 Commits (5) - [`65ec185`](https://github.com/netbirdio/netbird/commit/65ec1850afab1761664f73628f7c37e902996c4f) [management] Add structs for new networks concept (#3006) - [`99f785d`](https://github.com/netbirdio/netbird/commit/99f785dfb24758b069637180aa991b6b056b0da2) [management] Add name attr to network resource (#3011) - [`ea9841d`](https://github.com/netbirdio/netbird/commit/ea9841dd2f5b07873995ff007e27d733240591d1) [management] restructure api files (#3013) - [`c02918d`](https://github.com/netbirdio/netbird/commit/c02918d7c8dcdedfa78bb87e5503526c3822a83c) Add dns forwarder service - [`655891d`](https://github.com/netbirdio/netbird/commit/655891ddd4b03fd54ab6236c98e5c9729b783bf0) Filter out unmanaged domains ### 📊 Changes **41 files changed** (+1095 additions, -481 deletions) <details> <summary>View changed files</summary> ➕ `client/internal/dnsfwd/forwarder.go` (+115 -0) ➕ `client/internal/dnsfwd/manager.go` (+106 -0) 📝 `client/internal/engine.go` (+51 -12) 📝 `management/cmd/management.go` (+2 -1) 📝 `management/proto/management.pb.go` (+69 -43) 📝 `management/proto/management.proto` (+7 -0) ➕ `management/server/http/configs/auth.go` (+9 -0) 📝 `management/server/http/handler.go` (+21 -130) 📝 `management/server/http/handlers/accounts/accounts_handler.go` (+21 -13) 📝 `management/server/http/handlers/accounts/accounts_handler_test.go` (+6 -6) 📝 `management/server/http/handlers/dns/dns_settings_handler.go` (+23 -10) 📝 `management/server/http/handlers/dns/dns_settings_handler_test.go` (+5 -5) 📝 `management/server/http/handlers/dns/nameservers_handler.go` (+27 -17) 📝 `management/server/http/handlers/dns/nameservers_handler_test.go` (+7 -7) 📝 `management/server/http/handlers/events/events_handler.go` (+16 -9) 📝 `management/server/http/handlers/events/events_handler_test.go` (+5 -5) 📝 `management/server/http/handlers/groups/groups_handler.go` (+29 -18) 📝 `management/server/http/handlers/groups/groups_handler_test.go` (+9 -9) 📝 `management/server/http/handlers/peers/peers_handler.go` (+24 -15) 📝 `management/server/http/handlers/peers/peers_handler_test.go` (+3 -3) _...and 21 more files_ </details> ### 📄 Description ## Describe your changes ## Issue ticket number and link ### Checklist - [ ] Is it a bug fix - [ ] Is a typo/documentation fix - [ ] Is a feature enhancement - [ ] It is a refactor - [ ] Created tests that fail without the change (if possible) - [ ] Extended the README / documentation, if necessary --- <sub>🔄 This issue represents a GitHub Pull Request. It cannot be merged through Gitea due to API limitations.</sub>
saavagebueno added the pull-request label 2026-08-05 03:08:49 -04:00
Sign in to join this conversation.
No Label pull-request
1 Participants
Notifications
Due Date
No due date set.
Dependencies

No dependencies set.

Reference: DYNR/netbird#16029