Skip to content

Add disable letsencrypt#747

Merged
mlsmaycon merged 10 commits into
netbirdio:mainfrom
Ruakij:add-disable-letsencrypt
Apr 3, 2023
Merged

Add disable letsencrypt#747
mlsmaycon merged 10 commits into
netbirdio:mainfrom
Ruakij:add-disable-letsencrypt

Conversation

@Ruakij
Copy link
Copy Markdown
Contributor

@Ruakij Ruakij commented Mar 17, 2023

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?)

@CLAassistant
Copy link
Copy Markdown

CLAassistant commented Mar 17, 2023

CLA assistant check
All committers have signed the CLA.

Copy link
Copy Markdown
Collaborator

@mlsmaycon mlsmaycon left a comment

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

Thank you @Ruakij great addition to the project.

@mlsmaycon mlsmaycon merged commit 5993982 into netbirdio:main Apr 3, 2023
@Ruakij
Copy link
Copy Markdown
Contributor Author

Ruakij commented Apr 5, 2023

I have also marked the doc-changes as ready: netbirdio/docs#34

pulsastrix pushed a commit to pulsastrix/netbird that referenced this pull request Dec 24, 2023
Add NETBIRD_DISABLE_LETSENCRYPT support to explicit disable let's encrypt

Organize the setup.env.example variables into sections

Add traefik example
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Labels

None yet

Projects

None yet

Development

Successfully merging this pull request may close these issues.

3 participants