12583319996

Committed 02 Jan 2025 01:38PM UTC coverage: 57.522% (-1.1%) from 58.598%

Build # 12583319996

Build Type

Pull #9361

github

Committed by

starius

Commit Message

fn/ContextGuard: use context.AfterFunc to wait

Simplifies context cancellation handling by using context.AfterFunc instead of a
goroutine to wait for context cancellation. This approach avoids the overhead of
a goroutine during the waiting period.

For ctxQuitUnsafe, since g.quit is closed only in the Quit method (which also
cancels all associated contexts), waiting on context cancellation ensures the
same behavior without unnecessary dependency on g.quit.

Added a test to ensure that the Create method does not launch any goroutines.

Pull Request Pull Request #9361: fn: optimize context guard

Run Details

102587 of 178344 relevant lines covered (57.52%)

24734.33 hits per line

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

0.0

/lncfg/htlcswitch.go

package lncfg

import (
        "fmt"
        "time"
)

var (
        // MaxMailboxDeliveryTimeout specifies the max allowed timeout value.
        // This value is derived from the itest `async_bidirectional_payments`,
        // where both side send 483 payments at the same time to stress test
        // lnd.
        MaxMailboxDeliveryTimeout = 2 * time.Minute
)

//nolint:ll
type Htlcswitch struct {
        MailboxDeliveryTimeout time.Duration `long:"mailboxdeliverytimeout" description:"The timeout value when delivering HTLCs to a channel link. Setting this value too small will result in local payment failures if large number of payments are sent over a short period."`
}

// Validate checks the values configured for htlcswitch.
func (h *Htlcswitch) Validate() error {
        if h.MailboxDeliveryTimeout <= 0 {
                return fmt.Errorf("mailboxdeliverytimeout must be positive")
        }

        if h.MailboxDeliveryTimeout > MaxMailboxDeliveryTimeout {
                return fmt.Errorf("mailboxdeliverytimeout: %v exceeds "+
                        "maximum: %v", h.MailboxDeliveryTimeout,
                        MaxMailboxDeliveryTimeout)
        }

        return nil
}

1	package lncfg
2
3	import (
4	"fmt"
5	"time"
6	)
7
8	var (
9	// MaxMailboxDeliveryTimeout specifies the max allowed timeout value.
10	// This value is derived from the itest `async_bidirectional_payments`,
11	// where both side send 483 payments at the same time to stress test
12	// lnd.
13	MaxMailboxDeliveryTimeout = 2 * time.Minute
14	)
15
16	//nolint:ll
17	type Htlcswitch struct {
18	MailboxDeliveryTimeout time.Duration `long:"mailboxdeliverytimeout" description:"The timeout value when delivering HTLCs to a channel link. Setting this value too small will result in local payment failures if large number of payments are sent over a short period."`
19	}
20
21	// Validate checks the values configured for htlcswitch.
22	func (h *Htlcswitch) Validate() error {	×
23	if h.MailboxDeliveryTimeout <= 0 {	×
24	return fmt.Errorf("mailboxdeliverytimeout must be positive")	×
25	}	×
26
27	if h.MailboxDeliveryTimeout > MaxMailboxDeliveryTimeout {	×
28	return fmt.Errorf("mailboxdeliverytimeout: %v exceeds "+	×
29	"maximum: %v", h.MailboxDeliveryTimeout,	×
30	MaxMailboxDeliveryTimeout)	×
31	}	×
32
33	return nil	×
34	}

lightningnetwork / lnd / 12583319996

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