[PR #4724] [MERGED] [client] Add login_hint to oidc flows #26136

Closed
opened 2026-08-05 07:06:56 -04:00 by saavagebueno · 0 comments
Owner

📋 Pull Request Information

Original PR: https://github.com/netbirdio/netbird/pull/4724
Author: @lixmal
Created: 10/30/2025
Status: Merged
Merged: 11/5/2025
Merged by: @lixmal

Base: mainHead: login_hint


📝 Commits (6)

  • f7ad938 Add login_hint to oidc flows
  • d2ece2d [client] Use management-provided dns forwarder port on the client side (#4712)
  • 6db2f64 [client] Fix UI panic when switching profiles (#4718)
  • a1d27ab [client] Migrate deprecated grpc client code (#4687)
  • 0e2e4dd Pass hint to daemon rpc
  • 8c66593 Fix mobile build

📊 Changes

27 files changed (+193 additions, -48 deletions)

View changed files

📝 client/android/login.go (+1 -1)
📝 client/cmd/login.go (+21 -5)
📝 client/cmd/up.go (+8 -1)
📝 client/grpc/dialer.go (+35 -7)
📝 client/grpc/dialer_generic.go (+1 -2)
📝 client/internal/auth/device_flow.go (+25 -0)
📝 client/internal/auth/oauth.go (+11 -7)
📝 client/internal/auth/pkce_flow.go (+3 -0)
📝 client/internal/device_auth.go (+2 -0)
📝 client/internal/engine.go (+11 -1)
📝 client/internal/pkce_auth.go (+2 -0)
📝 client/internal/routemanager/common/params.go (+2 -0)
📝 client/internal/routemanager/dnsinterceptor/handler.go (+4 -2)
📝 client/internal/routemanager/manager.go (+11 -0)
📝 client/internal/routemanager/mock.go (+4 -0)
📝 client/ios/NetBirdSDK/client.go (+1 -1)
📝 client/net/conn.go (+7 -5)
📝 client/net/dial.go (+0 -1)
📝 client/net/dialer_dial.go (+2 -1)
📝 client/net/listener_listen.go (+2 -2)

...and 7 more files

📄 Description

Describe your changes

Stack

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)

By submitting this pull request, you confirm that you have read and agree to the terms of the Contributor License Agreement.

Documentation

Select exactly one:

  • I added/updated documentation for this change
  • Documentation is not needed for this change (explain why)

Docs PR URL (required if "docs added" is checked)

Paste the PR link from https://github.com/netbirdio/docs here:

https://github.com/netbirdio/docs/pull/__


🔄 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/4724 **Author:** [@lixmal](https://github.com/lixmal) **Created:** 10/30/2025 **Status:** ✅ Merged **Merged:** 11/5/2025 **Merged by:** [@lixmal](https://github.com/lixmal) **Base:** `main` ← **Head:** `login_hint` --- ### 📝 Commits (6) - [`f7ad938`](https://github.com/netbirdio/netbird/commit/f7ad9385fb209ef4ab0b3607e648b262ea4182f7) Add login_hint to oidc flows - [`d2ece2d`](https://github.com/netbirdio/netbird/commit/d2ece2d8288e6c437a644a13ba40739a051dc68e) [client] Use management-provided dns forwarder port on the client side (#4712) - [`6db2f64`](https://github.com/netbirdio/netbird/commit/6db2f64dbb250eec8cd848157fbf139ae3bd4168) [client] Fix UI panic when switching profiles (#4718) - [`a1d27ab`](https://github.com/netbirdio/netbird/commit/a1d27ab88eadae5cf8694fa628e2173a7cc38a75) [client] Migrate deprecated grpc client code (#4687) - [`0e2e4dd`](https://github.com/netbirdio/netbird/commit/0e2e4ddc610c98e7e59d5ea3f1bbb2c8addc4b8d) Pass hint to daemon rpc - [`8c66593`](https://github.com/netbirdio/netbird/commit/8c665936abf942de097bdcf02580b4facd0e4463) Fix mobile build ### 📊 Changes **27 files changed** (+193 additions, -48 deletions) <details> <summary>View changed files</summary> 📝 `client/android/login.go` (+1 -1) 📝 `client/cmd/login.go` (+21 -5) 📝 `client/cmd/up.go` (+8 -1) 📝 `client/grpc/dialer.go` (+35 -7) 📝 `client/grpc/dialer_generic.go` (+1 -2) 📝 `client/internal/auth/device_flow.go` (+25 -0) 📝 `client/internal/auth/oauth.go` (+11 -7) 📝 `client/internal/auth/pkce_flow.go` (+3 -0) 📝 `client/internal/device_auth.go` (+2 -0) 📝 `client/internal/engine.go` (+11 -1) 📝 `client/internal/pkce_auth.go` (+2 -0) 📝 `client/internal/routemanager/common/params.go` (+2 -0) 📝 `client/internal/routemanager/dnsinterceptor/handler.go` (+4 -2) 📝 `client/internal/routemanager/manager.go` (+11 -0) 📝 `client/internal/routemanager/mock.go` (+4 -0) 📝 `client/ios/NetBirdSDK/client.go` (+1 -1) 📝 `client/net/conn.go` (+7 -5) 📝 `client/net/dial.go` (+0 -1) 📝 `client/net/dialer_dial.go` (+2 -1) 📝 `client/net/listener_listen.go` (+2 -2) _...and 7 more files_ </details> ### 📄 Description ## Describe your changes ## Issue ticket number and link ## Stack <!-- branch-stack --> ### Checklist - [ ] Is it a bug fix - [ ] Is a typo/documentation fix - [x] Is a feature enhancement - [ ] It is a refactor - [ ] Created tests that fail without the change (if possible) > By submitting this pull request, you confirm that you have read and agree to the terms of the [Contributor License Agreement](https://github.com/netbirdio/netbird/blob/main/CONTRIBUTOR_LICENSE_AGREEMENT.md). ## Documentation Select exactly one: - [ ] I added/updated documentation for this change - [x] Documentation is **not needed** for this change (explain why) ### Docs PR URL (required if "docs added" is checked) Paste the PR link from https://github.com/netbirdio/docs here: https://github.com/netbirdio/docs/pull/__ --- <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 07:06:56 -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#26136