[client] Cache server key

[pappz]

Describe your changes

Cache management server key

Issue ticket number and link

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

[Copilot]

Pull Request Overview

This PR implements caching for the management server's public key to avoid repeated gRPC calls when the key is already known.

• Adds a srvKey field to the GrpcClient struct to store the cached server key
• Modifies GetServerPublicKey() to return the cached key if available, otherwise fetches and caches it

Comments suppressed due to low confidence (1)

shared/management/client/grpc.go:1

• The cached server key access and assignment is not thread-safe. Multiple goroutines calling GetServerPublicKey() concurrently could cause race conditions. Consider protecting the srvKey field with a mutex or using atomic operations.

package client

---

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

[sonarqubecloud]

Quality Gate passed

Issues
0 New issues
0 Accepted issues

Measures
0 Security Hotspots
0.0% Coverage on New Code
0.0% Duplication on New Code

See analysis details on SonarQube Cloud