[PR #1339] [MERGED] Fix the inability to set hostname with the flag in daemon mode #15253

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

📋 Pull Request Information

Original PR: https://github.com/netbirdio/netbird/pull/1339
Author: @bcmmbaga
Created: 11/29/2023
Status: Merged
Merged: 11/29/2023
Merged by: @mlsmaycon

Base: mainHead: fix/hostname-not-set-daemon-mode


📝 Commits (3)

  • 0754ff7 Add hostname to the LoginRequests and WaitSSOLoginRequests context
  • 2ad0bb5 Remove debug log statement
  • a2ae558 fix lint errors

📊 Changes

5 files changed (+160 additions, -124 deletions)

View changed files

📝 client/cmd/login.go (+2 -1)
📝 client/cmd/up.go (+2 -1)
📝 client/proto/daemon.pb.go (+142 -122)
📝 client/proto/daemon.proto (+3 -0)
📝 client/server/server.go (+11 -0)

📄 Description

Describe your changes

Pass the hostname set in the flag into the context when running in daemon mode.

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/1339 **Author:** [@bcmmbaga](https://github.com/bcmmbaga) **Created:** 11/29/2023 **Status:** ✅ Merged **Merged:** 11/29/2023 **Merged by:** [@mlsmaycon](https://github.com/mlsmaycon) **Base:** `main` ← **Head:** `fix/hostname-not-set-daemon-mode` --- ### 📝 Commits (3) - [`0754ff7`](https://github.com/netbirdio/netbird/commit/0754ff7a129d7894d004dc118a17b45685865a39) Add hostname to the LoginRequests and WaitSSOLoginRequests context - [`2ad0bb5`](https://github.com/netbirdio/netbird/commit/2ad0bb5e56111d2db1379326e831540c3f9cad07) Remove debug log statement - [`a2ae558`](https://github.com/netbirdio/netbird/commit/a2ae5582fc372e22790d3d7a0bed0cc18054274b) fix lint errors ### 📊 Changes **5 files changed** (+160 additions, -124 deletions) <details> <summary>View changed files</summary> 📝 `client/cmd/login.go` (+2 -1) 📝 `client/cmd/up.go` (+2 -1) 📝 `client/proto/daemon.pb.go` (+142 -122) 📝 `client/proto/daemon.proto` (+3 -0) 📝 `client/server/server.go` (+11 -0) </details> ### 📄 Description ## Describe your changes Pass the hostname set in the flag into the context when running in daemon mode. ## Issue ticket number and link - #1104 ### Checklist - [x] 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:07:16 -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#15253