[PR #4918] [CLOSED] Change NETBIRD_SIGNAL_PORT mapping from 80 to 10000 #21585

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

📋 Pull Request Information

Original PR: https://github.com/netbirdio/netbird/pull/4918
Author: @DaCaub
Created: 12/5/2025
Status: Closed

Base: mainHead: patch-1


📝 Commits (1)

  • 6b22c9b Change NETBIRD_SIGNAL_PORT mapping from 80 to 10000

📊 Changes

1 file changed (+1 additions, -1 deletions)

View changed files

📝 infrastructure_files/docker-compose.yml.tmpl (+1 -1)

📄 Description

This update aligns the template with the actual behavior of the Signal service and the official NetBird documentation.

Describe your changes

The advanced-setup docker-compose.yml template incorrectly mapped:

# Signal
  signal:
    ....
    ports:
  - $NETBIRD_SIGNAL_PORT:80

However, the netbird-signal container listens on port 10000 internally.

This mismatch caused external Signal connections to fail unless users manually adjusted their NAT/firewall rules or remapped the port.
The update changes the port mapping to:

# Signal
  signal:
    ....
    ports:
  - $NETBIRD_SIGNAL_PORT:10000

This ensures correct behavior out of the box and makes the template consistent with the actual service configuration and official documentation.

N/A — Bug discovered during advanced setup deployment.
(If an issue is later opened, it can be added here.)

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 (the fix corrects an incorrect port mapping in the docker template; no user-facing feature or behavior is changed, only the template is corrected to match the actual service port.)

Docs PR URL (required if "docs added" is checked)

Paste the PR link from https://github.com/netbirdio/docs here:

N/A

Summary by CodeRabbit

  • Chores
    • Updated Signal service port configuration in deployment setup.

✏️ Tip: You can customize this high-level summary in your review settings.


🔄 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/4918 **Author:** [@DaCaub](https://github.com/DaCaub) **Created:** 12/5/2025 **Status:** ❌ Closed **Base:** `main` ← **Head:** `patch-1` --- ### 📝 Commits (1) - [`6b22c9b`](https://github.com/netbirdio/netbird/commit/6b22c9b86190aad6a04074e0d49115a2d1c3c075) Change NETBIRD_SIGNAL_PORT mapping from 80 to 10000 ### 📊 Changes **1 file changed** (+1 additions, -1 deletions) <details> <summary>View changed files</summary> 📝 `infrastructure_files/docker-compose.yml.tmpl` (+1 -1) </details> ### 📄 Description This update aligns the template with the actual behavior of the Signal service and the official NetBird documentation. ## Describe your changes The advanced-setup docker-compose.yml template incorrectly mapped: ```yaml # Signal signal: .... ports: - $NETBIRD_SIGNAL_PORT:80 ``` However, the netbird-signal container listens on port 10000 internally. This mismatch caused external Signal connections to fail unless users manually adjusted their NAT/firewall rules or remapped the port. The update changes the port mapping to: ```yaml # Signal signal: .... ports: - $NETBIRD_SIGNAL_PORT:10000 ``` This ensures correct behavior out of the box and makes the template consistent with the actual service configuration and official documentation. ## Issue ticket number and link N/A — Bug discovered during advanced setup deployment. (If an issue is later opened, it can be added here.) ## Stack <!-- branch-stack --> ### Checklist - [x] 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](https://github.com/netbirdio/netbird/blob/main/CONTRIBUTOR_LICENSE_AGREEMENT.md). ## Documentation Select exactly one: - [ ] I added/updated documentation for this change - [x] Documentation is **not needed** for this change (the fix corrects an incorrect port mapping in the docker template; no user-facing feature or behavior is changed, only the template is corrected to match the actual service port.) ### Docs PR URL (required if "docs added" is checked) Paste the PR link from https://github.com/netbirdio/docs here: N/A <!-- This is an auto-generated comment: release notes by coderabbit.ai --> ## Summary by CodeRabbit * **Chores** * Updated Signal service port configuration in deployment setup. <sub>✏️ Tip: You can customize this high-level summary in your review settings.</sub> <!-- 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 05:08:41 -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#21585