[PR #5039] Fix: Signal container backend port set to right value #21885

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

📋 Pull Request Information

Original PR: https://github.com/netbirdio/netbird/pull/5039
Author: @x-504
Created: 1/6/2026
Status: 🔄 Open

Base: mainHead: fix_signal_port


📝 Commits (1)

  • 9e105bd Fix: Signal container backend port set to right value

📊 Changes

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

View changed files

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

📄 Description

Describe your changes

The backend port for the signal container was wrongly set to port 80 where the listenting port of the container is 10000 by default.

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.

Documentation

Select exactly one:

  • I added/updated documentation for this change
  • [ x] Documentation is not needed for this change (explain why)

Summary by CodeRabbit

  • Chores
    • Updated signal service port configuration to use environment variables for greater deployment flexibility, replacing the previously fixed port mapping.

✏️ 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/5039 **Author:** [@x-504](https://github.com/x-504) **Created:** 1/6/2026 **Status:** 🔄 Open **Base:** `main` ← **Head:** `fix_signal_port` --- ### 📝 Commits (1) - [`9e105bd`](https://github.com/netbirdio/netbird/commit/9e105bd1ee12ce02853090bcaa22ea4de29f0b8d) Fix: Signal container backend port set to right value ### 📊 Changes **1 file changed** (+2 additions, -2 deletions) <details> <summary>View changed files</summary> 📝 `infrastructure_files/docker-compose.yml.tmpl` (+2 -2) </details> ### 📄 Description ## Describe your changes The backend port for the signal container was wrongly set to port 80 where the listenting port of the container is 10000 by default. ## 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 (explain why) <!-- This is an auto-generated comment: release notes by coderabbit.ai --> ## Summary by CodeRabbit * **Chores** * Updated signal service port configuration to use environment variables for greater deployment flexibility, replacing the previously fixed port mapping. <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:09:04 -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#21885