[PR #628] [MERGED] Store the previous applied dns configuration hash #13906

Closed
opened 2026-08-05 02:09:10 -04:00 by saavagebueno · 0 comments
Owner

📋 Pull Request Information

Original PR: https://github.com/netbirdio/netbird/pull/628
Author: @mlsmaycon
Created: 12/16/2022
Status: Merged
Merged: 12/16/2022
Merged by: @mlsmaycon

Base: mainHead: hash-dns-configuration-change


📝 Commits (8)

  • 2996b0f Store the previous applied dns configuration hash
  • 74f63f6 sync go.mod
  • 2b12d0a debug docker
  • 1ca7598 change the testing domain
  • 1326eba remove secret dependency
  • 11e8f12 use env var for domain
  • 2a082a8 https when not localhost domain
  • d2f18ef if domain is localhost use default single acc mode domain

📊 Changes

6 files changed (+42 additions, -17 deletions)

View changed files

📝 .github/workflows/test-docker-compose-linux.yml (+7 -3)
📝 client/internal/dns/server.go (+30 -13)
📝 go.mod (+1 -0)
📝 go.sum (+2 -0)
📝 infrastructure_files/configure.sh (+1 -0)
📝 infrastructure_files/tests/setup.env (+1 -1)

📄 Description

This prevents changing the system DNS config when there is nothing to new

It also prevents issues with network changes on google chrome

Checklist

  • Is it a bug fix

🔄 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/628 **Author:** [@mlsmaycon](https://github.com/mlsmaycon) **Created:** 12/16/2022 **Status:** ✅ Merged **Merged:** 12/16/2022 **Merged by:** [@mlsmaycon](https://github.com/mlsmaycon) **Base:** `main` ← **Head:** `hash-dns-configuration-change` --- ### 📝 Commits (8) - [`2996b0f`](https://github.com/netbirdio/netbird/commit/2996b0f94540bc8ab8c9304bd30b35ffb72b5f51) Store the previous applied dns configuration hash - [`74f63f6`](https://github.com/netbirdio/netbird/commit/74f63f6cac9c93fe7a7d15dd013caad6e760cf8d) sync go.mod - [`2b12d0a`](https://github.com/netbirdio/netbird/commit/2b12d0aa7cb58ad2991b0e0a6b0442ba655c3884) debug docker - [`1ca7598`](https://github.com/netbirdio/netbird/commit/1ca7598141459d39f286af0922c3e744cb0b6246) change the testing domain - [`1326eba`](https://github.com/netbirdio/netbird/commit/1326ebaa8b9208cfdb7ccb95c77485d778cb5088) remove secret dependency - [`11e8f12`](https://github.com/netbirdio/netbird/commit/11e8f1226fdb171db9de82d7a1f688417496ae17) use env var for domain - [`2a082a8`](https://github.com/netbirdio/netbird/commit/2a082a8a795c856433ad5698725e4a2527d6bb6b) https when not localhost domain - [`d2f18ef`](https://github.com/netbirdio/netbird/commit/d2f18ef4c3d477fe5baed05222b6a24e6299fef7) if domain is localhost use default single acc mode domain ### 📊 Changes **6 files changed** (+42 additions, -17 deletions) <details> <summary>View changed files</summary> 📝 `.github/workflows/test-docker-compose-linux.yml` (+7 -3) 📝 `client/internal/dns/server.go` (+30 -13) 📝 `go.mod` (+1 -0) 📝 `go.sum` (+2 -0) 📝 `infrastructure_files/configure.sh` (+1 -0) 📝 `infrastructure_files/tests/setup.env` (+1 -1) </details> ### 📄 Description This prevents changing the system DNS config when there is nothing to new It also prevents issues with network changes on google chrome ## Issue ticket number and link ### Checklist - [x] Is it a bug fix --- <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 02:09:10 -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#13906