[PR #6786] [infrastructure] Fix remote NPM upstream configuration #28981

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

📋 Pull Request Information

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

Base: mainHead: fix/issue-5937-npm-host


📝 Commits (1)

  • 0ef633f [infrastructure] Fix remote NPM upstream configuration

📊 Changes

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

View changed files

📝 infrastructure_files/getting-started.sh (+24 -3)

📄 Description

Describe your changes

Fix the generated Nginx Proxy Manager configuration when NPM runs on a separate host. The quickstart script now asks for the reachable NetBird host and uses it in the generated proxy_pass, grpc_pass, and setup instructions.

When NPM shares a Docker network with NetBird, the existing container-name behavior is unchanged.

Fixes #5937

Stack

  • This PR is independent

Checklist

  • I have performed a self-review of my own code
  • I have tested the changes locally
  • I have added or updated tests where applicable
  • I have checked for breaking changes

Documentation

  • Documentation is not needed

Validation

  • bash -n infrastructure_files/getting-started.sh
  • git diff --check
  • ShellCheck was unavailable locally.

Summary by CodeRabbit

  • New Features
    • Added support for configuring Nginx Proxy Manager’s upstream host independently.
    • The setup process now prompts for an upstream host when required by the selected networking configuration.
    • Advanced configuration and post-setup guidance now reflect the configured upstream host.

🔄 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/6786 **Author:** [@CoderSufiyan](https://github.com/CoderSufiyan) **Created:** 7/15/2026 **Status:** 🔄 Open **Base:** `main` ← **Head:** `fix/issue-5937-npm-host` --- ### 📝 Commits (1) - [`0ef633f`](https://github.com/netbirdio/netbird/commit/0ef633fca6fb571100a753e87c733fbc32e25f2c) [infrastructure] Fix remote NPM upstream configuration ### 📊 Changes **1 file changed** (+24 additions, -3 deletions) <details> <summary>View changed files</summary> 📝 `infrastructure_files/getting-started.sh` (+24 -3) </details> ### 📄 Description ## Describe your changes Fix the generated Nginx Proxy Manager configuration when NPM runs on a separate host. The quickstart script now asks for the reachable NetBird host and uses it in the generated `proxy_pass`, `grpc_pass`, and setup instructions. When NPM shares a Docker network with NetBird, the existing container-name behavior is unchanged. ## Issue ticket number and link Fixes #5937 ## Stack - [x] This PR is independent ## Checklist - [x] I have performed a self-review of my own code - [x] I have tested the changes locally - [x] I have added or updated tests where applicable - [x] I have checked for breaking changes ## Documentation - [x] Documentation is **not needed** ## Validation - `bash -n infrastructure_files/getting-started.sh` - `git diff --check` - ShellCheck was unavailable locally. <!-- This is an auto-generated comment: release notes by coderabbit.ai --> ## Summary by CodeRabbit * **New Features** * Added support for configuring Nginx Proxy Manager’s upstream host independently. * The setup process now prompts for an upstream host when required by the selected networking configuration. * Advanced configuration and post-setup guidance now reflect the configured upstream host. <!-- 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 08:07:15 -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#28981