[PR #1534] [CLOSED] Feature/add sys meta netaddrs #13902

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

📋 Pull Request Information

Original PR: https://github.com/netbirdio/netbird/pull/1534
Author: @pappz
Created: 2/6/2024
Status: Closed

Base: feature/posture-checksHead: feature/add-sys-meta-netaddrs


📝 Commits (10+)

  • bb08dee Extract location by real ip as peer meta on sync
  • c29c150 Save peer location
  • 22493bf Test SavePeerLocation
  • 66044e9 Use net.IP type
  • 5747932 Remove TODO
  • d3c5d11 Fix linter
  • 740b64f Return geoname_id, public_ip and kernel_version as peer info
  • 80b812d Move Location out of Meta to Peer level
  • b80e257 Collect network addresses
  • 64f05e3 Add Linux sys product info

📊 Changes

32 files changed (+1891 additions, -2246 deletions)

View changed files

📝 client/system/info.go (+49 -0)
📝 client/system/info_darwin.go (+49 -1)
client/system/info_darwin_test.go (+23 -0)
📝 client/system/info_linux.go (+32 -4)
📝 client/system/info_test.go (+10 -0)
📝 client/system/info_windows.go (+76 -1)
client/system/info_windows_test.go (+23 -0)
📝 go.mod (+1 -0)
📝 go.sum (+2 -0)
📝 management/client/client_test.go (+11 -0)
📝 management/client/grpc.go (+10 -0)
📝 management/cmd/management.go (+1 -1)
📝 management/proto/management.pb.go (+1273 -2143)
📝 management/proto/management.proto (+9 -0)
📝 management/server/account.go (+1 -1)
📝 management/server/account_test.go (+3 -3)
📝 management/server/file_store.go (+21 -0)
📝 management/server/file_store_test.go (+49 -0)
📝 management/server/geolocation/geolocation.go (+2 -7)
📝 management/server/geolocation/geolocation_test.go (+2 -4)

...and 12 more files

📄 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/1534 **Author:** [@pappz](https://github.com/pappz) **Created:** 2/6/2024 **Status:** ❌ Closed **Base:** `feature/posture-checks` ← **Head:** `feature/add-sys-meta-netaddrs` --- ### 📝 Commits (10+) - [`bb08dee`](https://github.com/netbirdio/netbird/commit/bb08dee781fefe1b0e97b5f1d8a3debca77f696c) Extract location by real ip as peer meta on sync - [`c29c150`](https://github.com/netbirdio/netbird/commit/c29c150e061df8fb8b53eb76d06f339c89c2d9d0) Save peer location - [`22493bf`](https://github.com/netbirdio/netbird/commit/22493bfaedd97690f06a12f521eb3ee0cc1c1d52) Test SavePeerLocation - [`66044e9`](https://github.com/netbirdio/netbird/commit/66044e931fe224b02f1aac1f5d2b95684cedc160) Use net.IP type - [`5747932`](https://github.com/netbirdio/netbird/commit/5747932fc962412a801d1ef1b8aa208cafd8c00c) Remove TODO - [`d3c5d11`](https://github.com/netbirdio/netbird/commit/d3c5d112154a8a58613dd84d9575042998dc957b) Fix linter - [`740b64f`](https://github.com/netbirdio/netbird/commit/740b64fe58ca614acc8b15be8823e828877e06f5) Return geoname_id, public_ip and kernel_version as peer info - [`80b812d`](https://github.com/netbirdio/netbird/commit/80b812d330c47ce5e16ce071d49ed6f09fe79a57) Move Location out of Meta to Peer level - [`b80e257`](https://github.com/netbirdio/netbird/commit/b80e2579ae52cf1225779ddbc7a32441b04c9a1c) Collect network addresses - [`64f05e3`](https://github.com/netbirdio/netbird/commit/64f05e37b8db6f2bc0f9a04adeeed7b2070f62e1) Add Linux sys product info ### 📊 Changes **32 files changed** (+1891 additions, -2246 deletions) <details> <summary>View changed files</summary> 📝 `client/system/info.go` (+49 -0) 📝 `client/system/info_darwin.go` (+49 -1) ➕ `client/system/info_darwin_test.go` (+23 -0) 📝 `client/system/info_linux.go` (+32 -4) 📝 `client/system/info_test.go` (+10 -0) 📝 `client/system/info_windows.go` (+76 -1) ➕ `client/system/info_windows_test.go` (+23 -0) 📝 `go.mod` (+1 -0) 📝 `go.sum` (+2 -0) 📝 `management/client/client_test.go` (+11 -0) 📝 `management/client/grpc.go` (+10 -0) 📝 `management/cmd/management.go` (+1 -1) 📝 `management/proto/management.pb.go` (+1273 -2143) 📝 `management/proto/management.proto` (+9 -0) 📝 `management/server/account.go` (+1 -1) 📝 `management/server/account_test.go` (+3 -3) 📝 `management/server/file_store.go` (+21 -0) 📝 `management/server/file_store_test.go` (+49 -0) 📝 `management/server/geolocation/geolocation.go` (+2 -7) 📝 `management/server/geolocation/geolocation_test.go` (+2 -4) _...and 12 more files_ </details> ### 📄 Description ## Describe your changes ## Issue ticket number and link ### 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 --- <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 02:09:10 -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#13902