[PR #6737] [infrastructure] fix signal service port mapping for gRPC #26836

Open
opened 2026-08-05 07:07:52 -04:00 by saavagebueno · 0 comments
Owner

📋 Pull Request Information

Original PR: https://github.com/netbirdio/netbird/pull/6737
Author: @CoderSufiyan
Created: 7/12/2026
Status: 🔄 Open

Base: mainHead: fix/issue-4762-signal-port-mapping


📝 Commits (3)

  • 1e14822 [infrastructure] fix signal service port mapping for gRPC
  • 61a30b5 [infrastructure] Align Signal proxy guidance port
  • 1109600 [infrastructure] Update Signal port check

📊 Changes

3 files changed (+3 additions, -3 deletions)

View changed files

📝 .github/workflows/test-infrastructure-files.yml (+1 -1)
📝 infrastructure_files/configure.sh (+1 -1)
📝 infrastructure_files/docker-compose.yml.tmpl (+1 -1)

📄 Description

Description

The Docker Compose template mapped the signal service's external port to internal port 80, but the signal gRPC backward compatibility server runs on port 10000. This caused clients to see Signal as Disconnected when using custom TLS certificates.

Changes

  • Changed docker-compose.yml.tmpl signal port mapping from $NETBIRD_SIGNAL_PORT:80 to $NETBIRD_SIGNAL_PORT:10000

Documentation

  • Documentation is not needed for this change (infrastructure template fix)

Fixes #4762

Summary by CodeRabbit

  • Bug Fixes
    • Updated the Signal service container port mapping to use internal port 10000 instead of 80, ensuring gRPC traffic is routed correctly.
    • Adjusted the deployment/configuration checks to verify the Signal port mapping now matches 10000, improving connectivity consistency for configured environments.

🔄 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/6737 **Author:** [@CoderSufiyan](https://github.com/CoderSufiyan) **Created:** 7/12/2026 **Status:** 🔄 Open **Base:** `main` ← **Head:** `fix/issue-4762-signal-port-mapping` --- ### 📝 Commits (3) - [`1e14822`](https://github.com/netbirdio/netbird/commit/1e14822f5e28c85ade02c19c238aa8c325b552bd) [infrastructure] fix signal service port mapping for gRPC - [`61a30b5`](https://github.com/netbirdio/netbird/commit/61a30b598512ddee351432ebcaf6f3fedb7018b8) [infrastructure] Align Signal proxy guidance port - [`1109600`](https://github.com/netbirdio/netbird/commit/1109600b211021b254a963983b1155680e86b86b) [infrastructure] Update Signal port check ### 📊 Changes **3 files changed** (+3 additions, -3 deletions) <details> <summary>View changed files</summary> 📝 `.github/workflows/test-infrastructure-files.yml` (+1 -1) 📝 `infrastructure_files/configure.sh` (+1 -1) 📝 `infrastructure_files/docker-compose.yml.tmpl` (+1 -1) </details> ### 📄 Description ## Description The Docker Compose template mapped the signal service's external port to internal port 80, but the signal gRPC backward compatibility server runs on port 10000. This caused clients to see Signal as Disconnected when using custom TLS certificates. ## Changes - Changed docker-compose.yml.tmpl signal port mapping from `$NETBIRD_SIGNAL_PORT:80` to `$NETBIRD_SIGNAL_PORT:10000` ## Documentation - [x] Documentation is **not needed** for this change (infrastructure template fix) Fixes #4762 <!-- This is an auto-generated comment: release notes by coderabbit.ai --> ## Summary by CodeRabbit * **Bug Fixes** * Updated the Signal service container port mapping to use internal port **10000** instead of **80**, ensuring gRPC traffic is routed correctly. * Adjusted the deployment/configuration checks to verify the Signal port mapping now matches **10000**, improving connectivity consistency for configured environments. <!-- 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 07:07:52 -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#26836