[PR #2068] [MERGED] Generate network maps for routed network #14617

Closed
opened 2026-08-05 03:05:59 -04:00 by saavagebueno · 0 comments
Owner

📋 Pull Request Information

Original PR: https://github.com/netbirdio/netbird/pull/2068
Author: @bcmmbaga
Created: 5/28/2024
Status: Merged
Merged: 7/3/2024
Merged by: @bcmmbaga

Base: feature/network-route-access-controlHead: routed-net-network-maps


📝 Commits (10+)

  • 574dc50 extends route with access control groups
  • d6ab64c add support for creating and updating routes with access control groups
  • 00fd7d3 Add access control groups to routes API request and response
  • cda9c7e fix tests
  • 17f6984 fix tests
  • 79fcb04 Add network map processing for routed networks
  • 10ebf1c Refactor FirewallRule message and add RouteFirewallRule
  • a9cb906 Refactor enum and field names in management proto files
  • 8e60f79 Refactor firewall rules and add route firewall rules
  • 564482e Add firewall rules for routed networks

📊 Changes

13 files changed (+1730 additions, -580 deletions)

View changed files

📝 client/internal/acl/manager.go (+37 -37)
📝 client/internal/acl/manager_test.go (+72 -72)
📝 management/proto/management.pb.go (+813 -397)
📝 management/proto/management.proto (+76 -19)
📝 management/server/account.go (+8 -6)
📝 management/server/grpcserver.go (+13 -9)
📝 management/server/http/api/openapi.yml (+23 -1)
📝 management/server/http/api/types.gen.go (+21 -3)
📝 management/server/http/policies_handler.go (+32 -2)
📝 management/server/network.go (+7 -6)
📝 management/server/policy.go (+20 -28)
📝 management/server/route.go (+273 -0)
📝 management/server/route_test.go (+335 -0)

📄 Description

Describe your changes

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/2068 **Author:** [@bcmmbaga](https://github.com/bcmmbaga) **Created:** 5/28/2024 **Status:** ✅ Merged **Merged:** 7/3/2024 **Merged by:** [@bcmmbaga](https://github.com/bcmmbaga) **Base:** `feature/network-route-access-control` ← **Head:** `routed-net-network-maps` --- ### 📝 Commits (10+) - [`574dc50`](https://github.com/netbirdio/netbird/commit/574dc504cbc5a257b8ea157580b2667a6d966ab9) extends route with access control groups - [`d6ab64c`](https://github.com/netbirdio/netbird/commit/d6ab64c76dd72d8f6ca33e40ce2541586c032aaa) add support for creating and updating routes with access control groups - [`00fd7d3`](https://github.com/netbirdio/netbird/commit/00fd7d3e863aabf10bb2b9144404665e43fa258c) Add access control groups to routes API request and response - [`cda9c7e`](https://github.com/netbirdio/netbird/commit/cda9c7e47cd71134c994c7be828473127edf001b) fix tests - [`17f6984`](https://github.com/netbirdio/netbird/commit/17f6984896a4384fc0e3b9238a71f9bf9bef4c59) fix tests - [`79fcb04`](https://github.com/netbirdio/netbird/commit/79fcb04cdf56761fa01ba6a38b7737ebf8ef6002) Add network map processing for routed networks - [`10ebf1c`](https://github.com/netbirdio/netbird/commit/10ebf1c986e1073a69a3b5234b7c1b1286cefa7c) Refactor FirewallRule message and add RouteFirewallRule - [`a9cb906`](https://github.com/netbirdio/netbird/commit/a9cb9061624ec46ad155d5ec64c1230327a0d6c7) Refactor enum and field names in management proto files - [`8e60f79`](https://github.com/netbirdio/netbird/commit/8e60f79cabf83c0636917b6bd28fc6326b9a8bab) Refactor firewall rules and add route firewall rules - [`564482e`](https://github.com/netbirdio/netbird/commit/564482e32d44ae6608f07be51eab0acbd94ea3bf) Add firewall rules for routed networks ### 📊 Changes **13 files changed** (+1730 additions, -580 deletions) <details> <summary>View changed files</summary> 📝 `client/internal/acl/manager.go` (+37 -37) 📝 `client/internal/acl/manager_test.go` (+72 -72) 📝 `management/proto/management.pb.go` (+813 -397) 📝 `management/proto/management.proto` (+76 -19) 📝 `management/server/account.go` (+8 -6) 📝 `management/server/grpcserver.go` (+13 -9) 📝 `management/server/http/api/openapi.yml` (+23 -1) 📝 `management/server/http/api/types.gen.go` (+21 -3) 📝 `management/server/http/policies_handler.go` (+32 -2) 📝 `management/server/network.go` (+7 -6) 📝 `management/server/policy.go` (+20 -28) 📝 `management/server/route.go` (+273 -0) 📝 `management/server/route_test.go` (+335 -0) </details> ### 📄 Description ## Describe your changes ## Issue ticket number and link ### 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) - [ ] 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:05:59 -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#14617