Skip to content

Commit

Permalink
multi: use "bitcoin" chain name constant
Browse files Browse the repository at this point in the history
Make use of the `bitcoinChainName` constant instead of re-writing the
"bitcoin" string.
  • Loading branch information
ellemouton authored and yyforyongyu committed Sep 9, 2023
1 parent b9dc29d commit 548eaec
Show file tree
Hide file tree
Showing 4 changed files with 9 additions and 6 deletions.
5 changes: 3 additions & 2 deletions cmd/lncli/main.go
Original file line number Diff line number Diff line change
Expand Up @@ -15,6 +15,7 @@ import (
"syscall"

"github.com/btcsuite/btcd/btcutil"
"github.com/lightningnetwork/lnd"
"github.com/lightningnetwork/lnd/build"
"github.com/lightningnetwork/lnd/lncfg"
"github.com/lightningnetwork/lnd/lnrpc"
Expand Down Expand Up @@ -301,8 +302,8 @@ func extractPathArgs(ctx *cli.Context) (string, string, error) {
// lnddir/data/chain/<chain>/<network> in order to fetch the
// macaroon that we need.
macPath = filepath.Join(
lndDir, defaultDataDir, defaultChainSubDir, "bitcoin",
network, defaultMacaroonFilename,
lndDir, defaultDataDir, defaultChainSubDir,
lnd.BitcoinChainName, network, defaultMacaroonFilename,
)
}

Expand Down
4 changes: 2 additions & 2 deletions config.go
Original file line number Diff line number Diff line change
Expand Up @@ -259,7 +259,7 @@ var (
defaultBtcdDir = btcutil.AppDataDir(btcdBackendName, false)
defaultBtcdRPCCertFile = filepath.Join(defaultBtcdDir, "rpc.cert")

defaultBitcoindDir = btcutil.AppDataDir("bitcoin", false)
defaultBitcoindDir = btcutil.AppDataDir(BitcoinChainName, false)

defaultTorSOCKS = net.JoinHostPort("localhost", strconv.Itoa(defaultTorSOCKSPort))
defaultTorDNS = net.JoinHostPort(defaultTorDNSHost, strconv.Itoa(defaultTorDNSPort))
Expand Down Expand Up @@ -1808,7 +1808,7 @@ func parseRPCParams(cConfig *lncfg.Chain, nodeConfig interface{},
daemonName = bitcoindBackendName
confDir = conf.Dir
confFile = conf.ConfigPath
confFileBase = "bitcoin"
confFileBase = BitcoinChainName

// Check that cookie and credentials don't contradict each
// other.
Expand Down
3 changes: 2 additions & 1 deletion lntest/node/config.go
Original file line number Diff line number Diff line change
Expand Up @@ -11,6 +11,7 @@ import (
"sync/atomic"

"github.com/btcsuite/btcd/chaincfg"
"github.com/lightningnetwork/lnd"
"github.com/lightningnetwork/lnd/chanbackup"
"github.com/lightningnetwork/lnd/kvdb/etcd"
"github.com/lightningnetwork/lnd/lntest/wait"
Expand Down Expand Up @@ -157,7 +158,7 @@ func (cfg BaseNodeConfig) DBPath() string {

func (cfg BaseNodeConfig) ChanBackupPath() string {
return filepath.Join(
cfg.DataDir, "chain", "bitcoin",
cfg.DataDir, "chain", lnd.BitcoinChainName,
fmt.Sprintf(
"%v/%v", cfg.NetParams.Name,
chanbackup.DefaultBackupFileName,
Expand Down
3 changes: 2 additions & 1 deletion lntest/node/harness_node.go
Original file line number Diff line number Diff line change
Expand Up @@ -18,6 +18,7 @@ import (
"time"

"github.com/jackc/pgx/v4/pgxpool"
"github.com/lightningnetwork/lnd"
"github.com/lightningnetwork/lnd/lnrpc"
"github.com/lightningnetwork/lnd/lntest/rpc"
"github.com/lightningnetwork/lnd/lntest/wait"
Expand Down Expand Up @@ -107,7 +108,7 @@ func NewHarnessNode(t *testing.T, cfg *BaseNodeConfig) (*HarnessNode, error) {
cfg.TLSKeyPath = filepath.Join(cfg.BaseDir, "tls.key")

networkDir := filepath.Join(
cfg.DataDir, "chain", "bitcoin", cfg.NetParams.Name,
cfg.DataDir, "chain", lnd.BitcoinChainName, cfg.NetParams.Name,
)
cfg.AdminMacPath = filepath.Join(networkDir, "admin.macaroon")
cfg.ReadMacPath = filepath.Join(networkDir, "readonly.macaroon")
Expand Down

0 comments on commit 548eaec

Please sign in to comment.