Skip to content

Conversation

hakansa
Copy link
Member

@hakansa hakansa commented Aug 20, 2025

Ekran Resmi 2025-08-20 15 15 13 Ekran Resmi 2025-08-20 15 15 39

@Copilot Copilot AI review requested due to automatic review settings August 20, 2025 12:15
Copy link
Contributor

@Copilot Copilot AI left a comment

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

Pull Request Overview

This PR adds documentation for new exit node selection and auto-apply behavior features in NetBird. The changes explain how administrators can configure exit nodes with optional auto-apply settings and how client-side user preferences are handled.

  • Updates section title from "Routing Peer Selection" to "Exit node selection and auto-apply behavior"
  • Documents the "Skip Auto Apply" feature that allows exit nodes to be available without automatic enablement
  • Explains client override behavior where user selections take precedence over management preferences

Tip: Customize your code reviews with copilot-instructions.md. Create the file or learn how to get started.

### Exit Node Selection and Auto Apply Behavior

Currently, this is exclusively configured through the dashboard and cannot be influenced by the client.
Administrators configure exit nodes from the dashboard, and can optionally mark the default route (0.0.0.0/0) as selected by default. Clients will then auto-apply the selected exit node if the route is configured with **Auto Apply** or the user has made a local choice on the device.
Copy link

Copilot AI Aug 20, 2025

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

The sentence is missing a comma before 'and can optionally mark'. It should read: 'Administrators configure exit nodes from the dashboard, and can optionally mark the default route (0.0.0.0/0) as selected by default.'

Copilot uses AI. Check for mistakes.

…behavior, clarifying that routes created before the feature's introduction are treated as auto-applied by default.
@mlsmaycon mlsmaycon merged commit 69280e1 into main Aug 20, 2025
1 check passed
@mlsmaycon mlsmaycon deleted the feat/exit-node-autoapply branch August 20, 2025 14:46
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
None yet
Projects
None yet
Development

Successfully merging this pull request may close these issues.

2 participants