12293715361

Committed 12 Dec 2024 09:38AM UTC coverage: 57.483% (+7.9%) from 49.538%

Build # 12293715361

Build Type

Pull #9348

github

Committed by

ziggie1984

Commit Message

github: update goveralls tool

The goverall tool had a bug regarding the module versioning of
golang packages see also
https://github.com/mattn/goveralls/pull/222 for more background.
Goveralls is wrapped by another library to make it available for
github actions. So the relevant PR which is referenced here in
LND is:
https://github.com/shogo82148/actions-goveralls/pull/521.

Pull Request Pull Request #9348: github: update goveralls tool

Run Details

101897 of 177264 relevant lines covered (57.48%)

24982.4 hits per line

Source File
Press 'n' to go to next uncovered line, 'b' for previous

29.17

/lncfg/sweeper.go

package lncfg

import (
        "fmt"
        "time"

        "github.com/lightningnetwork/lnd/contractcourt"
        "github.com/lightningnetwork/lnd/lnwallet/chainfee"
        "github.com/lightningnetwork/lnd/sweep"
)

const (
        // MaxFeeRateFloor is the smallest config value allowed for the max fee
        // rate in sat/vb.
        MaxFeeRateFloor chainfee.SatPerVByte = 100

        // MaxAllowedFeeRate is the largest fee rate in sat/vb that we allow
        // when configuring the MaxFeeRate.
        MaxAllowedFeeRate = 10_000
)

//nolint:ll
type Sweeper struct {
        BatchWindowDuration time.Duration        `long:"batchwindowduration" description:"Duration of the sweep batch window. The sweep is held back during the batch window to allow more inputs to be added and thereby lower the fee per input." hidden:"true"`
        MaxFeeRate          chainfee.SatPerVByte `long:"maxfeerate" description:"Maximum fee rate in sat/vb that the sweeper is allowed to use when sweeping funds, the fee rate derived from budgets are capped at this value. Setting this value too low can result in transactions not being confirmed in time, causing HTLCs to expire hence potentially losing funds."`

        NoDeadlineConfTarget uint32 `long:"nodeadlineconftarget" description:"The conf target to use when sweeping non-time-sensitive outputs. This is useful for sweeping outputs that are not time-sensitive, and can be swept at a lower fee rate."`

        Budget *contractcourt.BudgetConfig `group:"sweeper.budget" namespace:"budget" long:"budget" description:"An optional config group that's used for the automatic sweep fee estimation. The Budget config gives options to limits ones fee exposure when sweeping unilateral close outputs and the fee rate calculated from budgets is capped at sweeper.maxfeerate. Check the budget config options for more details."`
}

// Validate checks the values configured for the sweeper.
func (s *Sweeper) Validate() error {
        if s.BatchWindowDuration < 0 {
                return fmt.Errorf("batchwindowduration must be positive")
        }

        // We require the max fee rate to be at least 100 sat/vbyte.
        if s.MaxFeeRate < MaxFeeRateFloor {
                return fmt.Errorf("maxfeerate must be >= 100 sat/vb")
        }

        // We require the max fee rate to be no greater than 10_000 sat/vbyte.
        if s.MaxFeeRate > MaxAllowedFeeRate {
                return fmt.Errorf("maxfeerate must be <= 10000 sat/vb")
        }

        // Make sure the conf target is at least 144 blocks (1 day).
        if s.NoDeadlineConfTarget < 144 {
                return fmt.Errorf("nodeadlineconftarget must be at least 144")
        }

        // Validate the budget configuration.
        if err := s.Budget.Validate(); err != nil {
                return fmt.Errorf("invalid budget config: %w", err)
        }

        return nil
}

// DefaultSweeperConfig returns the default configuration for the sweeper.
func DefaultSweeperConfig() *Sweeper {
        return &Sweeper{
                MaxFeeRate:           sweep.DefaultMaxFeeRate,
                NoDeadlineConfTarget: uint32(sweep.DefaultDeadlineDelta),
                Budget:               contractcourt.DefaultBudgetConfig(),
        }
}

1	package lncfg
2
3	import (
4	"fmt"
5	"time"
6
7	"github.com/lightningnetwork/lnd/contractcourt"
8	"github.com/lightningnetwork/lnd/lnwallet/chainfee"
9	"github.com/lightningnetwork/lnd/sweep"
10	)
11
12	const (
13	// MaxFeeRateFloor is the smallest config value allowed for the max fee
14	// rate in sat/vb.
15	MaxFeeRateFloor chainfee.SatPerVByte = 100
16
17	// MaxAllowedFeeRate is the largest fee rate in sat/vb that we allow
18	// when configuring the MaxFeeRate.
19	MaxAllowedFeeRate = 10_000
20	)
21
22	//nolint:ll
23	type Sweeper struct {
24	BatchWindowDuration time.Duration `long:"batchwindowduration" description:"Duration of the sweep batch window. The sweep is held back during the batch window to allow more inputs to be added and thereby lower the fee per input." hidden:"true"`
25	MaxFeeRate chainfee.SatPerVByte `long:"maxfeerate" description:"Maximum fee rate in sat/vb that the sweeper is allowed to use when sweeping funds, the fee rate derived from budgets are capped at this value. Setting this value too low can result in transactions not being confirmed in time, causing HTLCs to expire hence potentially losing funds."`
26
27	NoDeadlineConfTarget uint32 `long:"nodeadlineconftarget" description:"The conf target to use when sweeping non-time-sensitive outputs. This is useful for sweeping outputs that are not time-sensitive, and can be swept at a lower fee rate."`
28
29	Budget *contractcourt.BudgetConfig `group:"sweeper.budget" namespace:"budget" long:"budget" description:"An optional config group that's used for the automatic sweep fee estimation. The Budget config gives options to limits ones fee exposure when sweeping unilateral close outputs and the fee rate calculated from budgets is capped at sweeper.maxfeerate. Check the budget config options for more details."`
30	}
31
32	// Validate checks the values configured for the sweeper.
33	func (s *Sweeper) Validate() error {	×
34	if s.BatchWindowDuration < 0 {	×
35	return fmt.Errorf("batchwindowduration must be positive")	×
36	}	×
37
38	// We require the max fee rate to be at least 100 sat/vbyte.
39	if s.MaxFeeRate < MaxFeeRateFloor {	×
40	return fmt.Errorf("maxfeerate must be >= 100 sat/vb")	×
41	}	×
42
43	// We require the max fee rate to be no greater than 10_000 sat/vbyte.
44	if s.MaxFeeRate > MaxAllowedFeeRate {	×
45	return fmt.Errorf("maxfeerate must be <= 10000 sat/vb")	×
46	}	×
47
48	// Make sure the conf target is at least 144 blocks (1 day).
49	if s.NoDeadlineConfTarget < 144 {	×
50	return fmt.Errorf("nodeadlineconftarget must be at least 144")	×
51	}	×
52
53	// Validate the budget configuration.
54	if err := s.Budget.Validate(); err != nil {	×
55	return fmt.Errorf("invalid budget config: %w", err)	×
56	}	×
57
58	return nil	×
59	}
60
61	// DefaultSweeperConfig returns the default configuration for the sweeper.
62	func DefaultSweeperConfig() *Sweeper {	1✔
63	return &Sweeper{	1✔
64	MaxFeeRate: sweep.DefaultMaxFeeRate,	1✔
65	NoDeadlineConfTarget: uint32(sweep.DefaultDeadlineDelta),	1✔
66	Budget: contractcourt.DefaultBudgetConfig(),	1✔
67	}	1✔
68	}	1✔

lightningnetwork / lnd / 12293715361

Source File Press 'n' to go to next uncovered line, 'b' for previous

Source File
Press 'n' to go to next uncovered line, 'b' for previous