[PR #6769] [MERGED] [management,client] 0.74.5 branch sync #29878

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

📋 Pull Request Information

Original PR: https://github.com/netbirdio/netbird/pull/6769
Author: @mlsmaycon
Created: 7/14/2026
Status: Merged
Merged: 7/14/2026
Merged by: @mlsmaycon

Base: mainHead: 0.74.5-branch-sync


📝 Commits (8)

  • 816d806 [client] Update gopsutil to v4 (#6688)
  • 7cd5c17 [client] Fix hanging status command during relay dial (#6694)
  • 08e46aa [management] fix: prevent reverse proxy domain from being pushed as DNS search domain (#6498)
  • 4d4cc55 [client] Recover from rosenpass key desync (#6714)
  • 3d87547 [client] Bump golang.org/x/crypto to v0.54.0 and Go toolchain to 1.25.12 (#6709)
  • 277d8e4 [proxy] enforce model allowlist for URL-routed providers (Bedrock/Vertex) (#6764)
  • f0eed75 [management] Remove proxy peer stale deduplication logic (#6768)
  • f28ff2a Merge branch 'main' into 0.74.5-branch-sync

📊 Changes

9 files changed (+369 additions, -261 deletions)

View changed files

📝 e2e/agentnetwork/chat_test.go (+6 -3)
e2e/agentnetwork/guardrail_test.go (+168 -0)
📝 e2e/harness/agentnetwork.go (+11 -0)
📝 e2e/harness/client.go (+16 -0)
📝 management/internals/modules/peers/manager.go (+0 -50)
management/server/agentnetwork_proxypeer_restart_test.go (+0 -199)
📝 proxy/internal/middleware/builtin/llm_guardrail/middleware.go (+28 -6)
📝 proxy/internal/middleware/builtin/llm_guardrail/middleware_test.go (+34 -3)
proxy/internal/middleware/builtin/llm_request_parser/guardrail_allowlist_test.go (+106 -0)

📄 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)
  • This change does not modify the public API, gRPC protocols, functionality behavior, CLI / service flags, or introduce a new feature — OR I have discussed it with the NetBird team beforehand (link the issue / Slack thread in the description). See CONTRIBUTING.md.

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

  • New Features

    • Added model-allowlist guardrails for path-routed providers, including Bedrock and Vertex.
    • Added Bedrock request support for chat interactions.
    • Added guardrail management capabilities.
  • Bug Fixes

    • Requests with missing or blank model identifiers are now denied when a model allowlist is configured, improving fail-closed protection.
    • Corrected provider-specific request handling and session tracking for Bedrock interactions.
  • Tests

    • Expanded coverage for allowlist enforcement and provider routing scenarios.

🔄 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/6769 **Author:** [@mlsmaycon](https://github.com/mlsmaycon) **Created:** 7/14/2026 **Status:** ✅ Merged **Merged:** 7/14/2026 **Merged by:** [@mlsmaycon](https://github.com/mlsmaycon) **Base:** `main` ← **Head:** `0.74.5-branch-sync` --- ### 📝 Commits (8) - [`816d806`](https://github.com/netbirdio/netbird/commit/816d80602ffc024afb670dfe18766092701d2639) [client] Update gopsutil to v4 (#6688) - [`7cd5c17`](https://github.com/netbirdio/netbird/commit/7cd5c1732bb5374f21005073937c42f4d531e3c5) [client] Fix hanging status command during relay dial (#6694) - [`08e46aa`](https://github.com/netbirdio/netbird/commit/08e46aa62f3da6a4516dac68287084b736790214) [management] fix: prevent reverse proxy domain from being pushed as DNS search domain (#6498) - [`4d4cc55`](https://github.com/netbirdio/netbird/commit/4d4cc551fdd0b8824ae4caf44af6586d43c044df) [client] Recover from rosenpass key desync (#6714) - [`3d87547`](https://github.com/netbirdio/netbird/commit/3d87547d952f5ada9df987bbe4f0f6d54372d77c) [client] Bump golang.org/x/crypto to v0.54.0 and Go toolchain to 1.25.12 (#6709) - [`277d8e4`](https://github.com/netbirdio/netbird/commit/277d8e4c5352950e1ec4fbd21a3266f0412b09fe) [proxy] enforce model allowlist for URL-routed providers (Bedrock/Vertex) (#6764) - [`f0eed75`](https://github.com/netbirdio/netbird/commit/f0eed7564f3a9138962da1408986e4666d7137b5) [management] Remove proxy peer stale deduplication logic (#6768) - [`f28ff2a`](https://github.com/netbirdio/netbird/commit/f28ff2a1f1f65971598111626160889082bfbaeb) Merge branch 'main' into 0.74.5-branch-sync ### 📊 Changes **9 files changed** (+369 additions, -261 deletions) <details> <summary>View changed files</summary> 📝 `e2e/agentnetwork/chat_test.go` (+6 -3) ➕ `e2e/agentnetwork/guardrail_test.go` (+168 -0) 📝 `e2e/harness/agentnetwork.go` (+11 -0) 📝 `e2e/harness/client.go` (+16 -0) 📝 `management/internals/modules/peers/manager.go` (+0 -50) ➖ `management/server/agentnetwork_proxypeer_restart_test.go` (+0 -199) 📝 `proxy/internal/middleware/builtin/llm_guardrail/middleware.go` (+28 -6) 📝 `proxy/internal/middleware/builtin/llm_guardrail/middleware_test.go` (+34 -3) ➕ `proxy/internal/middleware/builtin/llm_request_parser/guardrail_allowlist_test.go` (+106 -0) </details> ### 📄 Description ## Describe your changes * [proxy] enforce model allowlist for URL-routed providers (Bedrock/Vertex) by @mlsmaycon in https://github.com/netbirdio/netbird/pull/6764 * [management] Remove proxy peer stale deduplication logic by @mlsmaycon in https://github.com/netbirdio/netbird/pull/6768 ## Issue ticket number and link ## Stack <!-- branch-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) - [ ] This change does **not** modify the public API, gRPC protocols, functionality behavior, CLI / service flags, or introduce a new feature — **OR** I have discussed it with the NetBird team beforehand (link the issue / Slack thread in the description). See [CONTRIBUTING.md](https://github.com/netbirdio/netbird/blob/main/CONTRIBUTING.md#discuss-changes-with-the-netbird-team-first). > 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 - **New Features** - Added model-allowlist guardrails for path-routed providers, including Bedrock and Vertex. - Added Bedrock request support for chat interactions. - Added guardrail management capabilities. - **Bug Fixes** - Requests with missing or blank model identifiers are now denied when a model allowlist is configured, improving fail-closed protection. - Corrected provider-specific request handling and session tracking for Bedrock interactions. - **Tests** - Expanded coverage for allowlist enforcement and provider routing scenarios. <!-- end of auto-generated comment: release notes by coderabbit.ai --> --- <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 08:09: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#29878