aiolabs/nix-bitcoin
1
0
Fork 0
Code Issues Pull requests Projects Releases Packages Wiki Activity Actions 1

lamassu: clarify hostname option and add TODO for certificate redundancy

Browse source 
The hostname is embedded in the pairing QR code and tells ATMs where to
connect. Added a TODO to consider auto-populating certificate.extraIPs
from hostname when it's an IP address.

🤖 Generated with [Claude Code](https://claude.com/claude-code)

Co-Authored-By: Claude Opus 4.5 <noreply@anthropic.com>
This commit is contained in:
padreug 2025-12-23 13:09:06 +01:00
parent a7d96d2d2e
commit 8ee71833b9
1 changed files with 7 additions and 1 deletions
Download patch file Download diff file Expand all files Collapse all files

8
modules/lamassu-lnbits.nix
View file

@ -114,10 +114,16 @@ in
hostname = mkOption {
type = types.str;
default = "localhost";
description = "Hostname for the server (used by application)";
description = ''
Hostname for the server. This is embedded in the pairing QR code
and tells ATMs where to connect. Can be an IP address or domain name.
'';
};
# Certificate options (same pattern as LND)
# TODO: When using an IP address, hostname and certificate.extraIPs are redundant.
# Consider auto-populating certificate.extraIPs from hostname if it's an IP,
# or unifying these options. For now, set both to the same IP address.
certificate = {
extraIPs = mkOption {
type = with types; listOf str;