[PR #747] [MERGED] Add disable letsencrypt #12950

Closed
opened 2026-08-05 02:07:05 -04:00 by saavagebueno · 0 comments
Owner

📋 Pull Request Information

Original PR: https://github.com/netbirdio/netbird/pull/747
Author: @Ruakij
Created: 3/17/2023
Status: Merged
Merged: 4/3/2023
Merged by: @mlsmaycon

Base: mainHead: add-disable-letsencrypt


📝 Commits (10+)

  • 820b65a Reformat for better readability
  • fd5152f Add env-option to disable usage of letsencrypt
  • 55bef66 Add info about more endpoints and grpc protocol
  • 454e58b Add variable for signalexchange port
  • f7a73ca Add variable signal-protocol for manipulating
  • a46370b Move letsencrypt check to bottom
  • 9f0b18a Add traefik template-file
  • 04faf3f Fix signal-env-vars
  • 276a3d5 Add full prefix-path
  • d2b83e8 Unset LETSENCRYPT_DOMAIN instead of setting to none

📊 Changes

6 files changed (+174 additions, -17 deletions)

View changed files

📝 infrastructure_files/base.setup.env (+10 -4)
📝 infrastructure_files/configure.sh (+26 -0)
📝 infrastructure_files/docker-compose.yml.tmpl (+22 -8)
infrastructure_files/docker-compose.yml.tmpl.traefik (+99 -0)
📝 infrastructure_files/management.json.tmpl (+2 -2)
📝 infrastructure_files/setup.env.example (+15 -3)

📄 Description

This PR adds the setup-option NETBIRD_DISABLE_LETSENCRYPT in setup.env.example to allow disabling the internal letsencrypt-setup.
e.g. for use with a reverse-proxy or to ship own letsencrypt-certs from an external-system.

For this i also had to add some env-setup-variables.
I also reordered the compose aswell as the setup.env file to make it more readable.

When running configure with NETBIRD_DISABLE_LETSENCRYPT=true the user is informed about actions he must take to get a working setup.

I have also prepared doc-changes to include more extensive documentation about using netbird behind a reverse-proxy and shipped an example-file for traefik.
I plan to submit that PR when this one is accepted.
EDIT: PR for docs

But I am not 100% happy with how much the user has to edit to archive such a setup, but i think the example-files will help alot.
(Maybe combine the domains and ports, so only 1 thing has to be changed?)


🔄 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/747 **Author:** [@Ruakij](https://github.com/Ruakij) **Created:** 3/17/2023 **Status:** ✅ Merged **Merged:** 4/3/2023 **Merged by:** [@mlsmaycon](https://github.com/mlsmaycon) **Base:** `main` ← **Head:** `add-disable-letsencrypt` --- ### 📝 Commits (10+) - [`820b65a`](https://github.com/netbirdio/netbird/commit/820b65a4d475e29cf58686fb6c86e33a2378d172) Reformat for better readability - [`fd5152f`](https://github.com/netbirdio/netbird/commit/fd5152f0f759ce23b8c52904648674ed734c5ff7) Add env-option to disable usage of letsencrypt - [`55bef66`](https://github.com/netbirdio/netbird/commit/55bef66ef9e4bb9526ad408f08730f4cd43cd4c1) Add info about more endpoints and grpc protocol - [`454e58b`](https://github.com/netbirdio/netbird/commit/454e58b0bc67a1bd52854a7edefd6f66af8f8342) Add variable for signalexchange port - [`f7a73ca`](https://github.com/netbirdio/netbird/commit/f7a73ca2e073bbade2ea11ec58a9a3230d6b107d) Add variable signal-protocol for manipulating - [`a46370b`](https://github.com/netbirdio/netbird/commit/a46370b9747c30976278aa0f8a2346bd5eb8769d) Move letsencrypt check to bottom - [`9f0b18a`](https://github.com/netbirdio/netbird/commit/9f0b18a33044410634f9cf2d5e78d427ae3a67d9) Add traefik template-file - [`04faf3f`](https://github.com/netbirdio/netbird/commit/04faf3f7c673ad9ba826b58424eb2013d8b3cb80) Fix signal-env-vars - [`276a3d5`](https://github.com/netbirdio/netbird/commit/276a3d5f11af597e0c2ac2f149b0a35ab8792973) Add full prefix-path - [`d2b83e8`](https://github.com/netbirdio/netbird/commit/d2b83e882d34caf47eed6e8e2f376a7cd197c0ee) Unset LETSENCRYPT_DOMAIN instead of setting to none ### 📊 Changes **6 files changed** (+174 additions, -17 deletions) <details> <summary>View changed files</summary> 📝 `infrastructure_files/base.setup.env` (+10 -4) 📝 `infrastructure_files/configure.sh` (+26 -0) 📝 `infrastructure_files/docker-compose.yml.tmpl` (+22 -8) ➕ `infrastructure_files/docker-compose.yml.tmpl.traefik` (+99 -0) 📝 `infrastructure_files/management.json.tmpl` (+2 -2) 📝 `infrastructure_files/setup.env.example` (+15 -3) </details> ### 📄 Description This PR adds the setup-option `NETBIRD_DISABLE_LETSENCRYPT` in `setup.env.example` to allow disabling the internal letsencrypt-setup. e.g. for use with a reverse-proxy or to ship own letsencrypt-certs from an external-system. For this i also had to add some env-setup-variables. I also reordered the compose aswell as the setup.env file to make it more readable. When running configure with `NETBIRD_DISABLE_LETSENCRYPT=true` the user is informed about actions he must take to get a working setup. I have also prepared doc-changes to include more extensive documentation about using netbird behind a reverse-proxy and shipped an example-file for traefik. ~~I plan to submit that PR when this one is accepted.~~ EDIT: [PR for docs](https://github.com/netbirdio/docs/pull/34) But I am not 100% happy with how much the user has to edit to archive such a setup, but i think the example-files will help alot. (Maybe combine the domains and ports, so only 1 thing has to be changed?) --- <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 02:07:05 -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#12950