[PR #4807] [management] pass config to controller #4413

Open
opened 2025-11-20 08:08:20 -05:00 by saavagebueno · 0 comments
Owner

Original Pull Request: https://github.com/netbirdio/netbird/pull/4807

State: closed
Merged: Yes


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/__

Summary by CodeRabbit

  • Refactor
    • Sync behavior now sources HTTP and device-authorization settings from the central server configuration.
    • Sync payloads and peer updates include HTTP and device-authorization parameters so devices receive auth/HTTP data from server config.
    • Controller initialization now injects global configuration into update flows, making sync responses config-driven.
    • NetbirdConfig no longer exposes an embedded JWT config in sync payloads.
**Original Pull Request:** https://github.com/netbirdio/netbird/pull/4807 **State:** closed **Merged:** Yes --- ## Describe your changes ## Issue ticket number and link ## Stack <!-- branch-stack --> ### 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) > 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/__ <!-- This is an auto-generated comment: release notes by coderabbit.ai --> ## Summary by CodeRabbit * **Refactor** * Sync behavior now sources HTTP and device-authorization settings from the central server configuration. * Sync payloads and peer updates include HTTP and device-authorization parameters so devices receive auth/HTTP data from server config. * Controller initialization now injects global configuration into update flows, making sync responses config-driven. * NetbirdConfig no longer exposes an embedded JWT config in sync payloads. <!-- end of auto-generated comment: release notes by coderabbit.ai -->
saavagebueno added the pull-request label 2025-11-20 08:08:20 -05:00
Sign in to join this conversation.
1 Participants
Notifications
Due Date
No due date set.
Dependencies

No dependencies set.

Reference: SVI/netbird#4413