[PR #62] [MERGED] update signal gRpc, enable TLS and add keepalive params #2511

Closed
opened 2025-11-20 07:11:20 -05:00 by saavagebueno · 0 comments
Owner

📋 Pull Request Information

Original PR: https://github.com/netbirdio/netbird/pull/62
Author: @braginini
Created: 7/21/2021
Status: Merged
Merged: 7/21/2021
Merged by: @braginini

Base: mainHead: update-signal-grpc


📝 Commits (5)

  • 46021c9 chore: update signal gRpc
  • 4671d6b chore: add Signal keep alive params and policy
  • 7c4d6b7 feature: add signal TLS support
  • 79c512d fix: use signal commands in signal cmd
  • ff02ac5 refactor: move signal Dockerfile to the corresponding folder

📊 Changes

9 files changed (+555 additions, -319 deletions)

View changed files

📝 .goreleaser.yaml (+2 -2)
📝 cmd/root.go (+33 -0)
📝 cmd/signal.go (+39 -4)
📝 signal/Dockerfile (+0 -2)
📝 signal/README.md (+9 -10)
📝 signal/proto/signalexchange.pb.go (+296 -300)
📝 signal/proto/signalexchange.proto (+1 -1)
signal/proto/signalexchange_grpc.pb.go (+174 -0)
📝 signal/signal.go (+1 -0)

📄 Description

  • update to protoc-gen-go-grpc
  • add keepalive params
  • add TLS (letsencrypt)

🔄 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/62 **Author:** [@braginini](https://github.com/braginini) **Created:** 7/21/2021 **Status:** ✅ Merged **Merged:** 7/21/2021 **Merged by:** [@braginini](https://github.com/braginini) **Base:** `main` ← **Head:** `update-signal-grpc` --- ### 📝 Commits (5) - [`46021c9`](https://github.com/netbirdio/netbird/commit/46021c9b1c2dd24b573eccab4ea4fc8bfc160ba1) chore: update signal gRpc - [`4671d6b`](https://github.com/netbirdio/netbird/commit/4671d6b33c75cf7362dc52357e8292847d63d5c4) chore: add Signal keep alive params and policy - [`7c4d6b7`](https://github.com/netbirdio/netbird/commit/7c4d6b7860d04e0098ac1192a72393c2134db802) feature: add signal TLS support - [`79c512d`](https://github.com/netbirdio/netbird/commit/79c512db2c85fbb9760b2c1122c68691873ebba6) fix: use signal commands in signal cmd - [`ff02ac5`](https://github.com/netbirdio/netbird/commit/ff02ac59004527887a2b19aaf421b000557d0698) refactor: move signal Dockerfile to the corresponding folder ### 📊 Changes **9 files changed** (+555 additions, -319 deletions) <details> <summary>View changed files</summary> 📝 `.goreleaser.yaml` (+2 -2) 📝 `cmd/root.go` (+33 -0) 📝 `cmd/signal.go` (+39 -4) 📝 `signal/Dockerfile` (+0 -2) 📝 `signal/README.md` (+9 -10) 📝 `signal/proto/signalexchange.pb.go` (+296 -300) 📝 `signal/proto/signalexchange.proto` (+1 -1) ➕ `signal/proto/signalexchange_grpc.pb.go` (+174 -0) 📝 `signal/signal.go` (+1 -0) </details> ### 📄 Description - [x] update to protoc-gen-go-grpc - [x] add keepalive params - [x] add TLS (letsencrypt) --- <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 2025-11-20 07:11: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#2511