[PR #2257] Support custom SSL certificates for the signal service #14873

Open
opened 2026-08-05 03:06:30 -04:00 by saavagebueno · 0 comments
Owner

Original Pull Request: https://github.com/netbirdio/netbird/pull/2257

State: closed
Merged: Yes


Describe your changes

This PR adds the ability to provide custom SSL certificates to the signal service. Most of the code was adapted from management/cmd/management.go since it already offers the functionality I was looking to implement.

Two flags have been added, --cert-file and --cert-key, which accept a string pointing to a file path for the SSL certificate and key. When --letsencrypt-domain is set, these options are ignored.

This is my first try at go, so please let me know if anything needs refactored and fixed!

Along with https://github.com/netbirdio/dashboard/pull/399, closes #2242

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)
  • Extended the README / documentation, if necessary
**Original Pull Request:** https://github.com/netbirdio/netbird/pull/2257 **State:** closed **Merged:** Yes --- ## Describe your changes This PR adds the ability to provide custom SSL certificates to the signal service. Most of the code was adapted from [management/cmd/management.go](https://github.com/netbirdio/netbird/blob/e8c2fafccd9ebb649a3d15f1b361e09bb2b61639/management/cmd/management.go) since it already offers the functionality I was looking to implement. Two flags have been added, `--cert-file` and `--cert-key`, which accept a string pointing to a file path for the SSL certificate and key. When `--letsencrypt-domain` is set, these options are ignored. This is my first try at go, so please let me know if anything needs refactored and fixed! ## Issue ticket number and link Along with https://github.com/netbirdio/dashboard/pull/399, closes #2242 ### Checklist - [ ] Is it a bug fix - [ ] Is a typo/documentation fix - [x] Is a feature enhancement - [x] It is a refactor - [ ] Created tests that fail without the change (if possible) - [x] Extended the README / documentation, if necessary
saavagebueno added the pull-request label 2026-08-05 03:06:30 -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#14873