routing+server: let ForAllOutgoingChannels take a context
And so, remove a previously added context.TODO
This commit is contained in:
3 files changed
+10
-9
No files matched your search
@@ -40,8 +40,9 @@ type Manager struct {
|
|||||||
|
|
||||||
// ForAllOutgoingChannels is required to iterate over all our local
|
// ForAllOutgoingChannels is required to iterate over all our local
|
||||||
// channels. The ChannelEdgePolicy parameter may be nil.
|
// channels. The ChannelEdgePolicy parameter may be nil.
|
||||||
ForAllOutgoingChannels func(cb func(*models.ChannelEdgeInfo,
|
ForAllOutgoingChannels func(ctx context.Context,
|
||||||
*models.ChannelEdgePolicy) error) error
|
cb func(*models.ChannelEdgeInfo,
|
||||||
|
*models.ChannelEdgePolicy) error) error
|
||||||
|
|
||||||
// FetchChannel is used to query local channel parameters. Optionally an
|
// FetchChannel is used to query local channel parameters. Optionally an
|
||||||
// existing db tx can be supplied.
|
// existing db tx can be supplied.
|
||||||
@@ -151,7 +152,7 @@ func (r *Manager) UpdatePolicy(ctx context.Context,
|
|||||||
// Next, we'll loop over all the outgoing channels the router knows of.
|
// Next, we'll loop over all the outgoing channels the router knows of.
|
||||||
// If we have a filter then we'll only collect those channels, otherwise
|
// If we have a filter then we'll only collect those channels, otherwise
|
||||||
// we'll collect them all.
|
// we'll collect them all.
|
||||||
err := r.ForAllOutgoingChannels(processChan)
|
err := r.ForAllOutgoingChannels(ctx, processChan)
|
||||||
if err != nil {
|
if err != nil {
|
||||||
return nil, err
|
return nil, err
|
||||||
}
|
}
|
||||||
|
|||||||
@@ -123,8 +123,9 @@ func TestManager(t *testing.T) {
|
|||||||
return nil
|
return nil
|
||||||
}
|
}
|
||||||
|
|
||||||
forAllOutgoingChannels := func(cb func(*models.ChannelEdgeInfo,
|
forAllOutgoingChannels := func(_ context.Context,
|
||||||
*models.ChannelEdgePolicy) error) error {
|
cb func(*models.ChannelEdgeInfo,
|
||||||
|
*models.ChannelEdgePolicy) error) error {
|
||||||
|
|
||||||
for _, c := range channelSet {
|
for _, c := range channelSet {
|
||||||
if err := cb(c.edgeInfo, ¤tPolicy); err != nil {
|
if err := cb(c.edgeInfo, ¤tPolicy); err != nil {
|
||||||
|
|||||||
@@ -1252,10 +1252,9 @@ func newServer(ctx context.Context, cfg *Config, listenAddrs []net.Addr,
|
|||||||
s.localChanMgr = &localchans.Manager{
|
s.localChanMgr = &localchans.Manager{
|
||||||
SelfPub: nodeKeyDesc.PubKey,
|
SelfPub: nodeKeyDesc.PubKey,
|
||||||
DefaultRoutingPolicy: cc.RoutingPolicy,
|
DefaultRoutingPolicy: cc.RoutingPolicy,
|
||||||
ForAllOutgoingChannels: func(cb func(*models.ChannelEdgeInfo,
|
ForAllOutgoingChannels: func(ctx context.Context,
|
||||||
*models.ChannelEdgePolicy) error) error {
|
cb func(*models.ChannelEdgeInfo,
|
||||||
|
*models.ChannelEdgePolicy) error) error {
|
||||||
ctx := context.TODO()
|
|
||||||
|
|
||||||
return s.graphDB.ForEachNodeChannel(ctx, selfVertex,
|
return s.graphDB.ForEachNodeChannel(ctx, selfVertex,
|
||||||
func(c *models.ChannelEdgeInfo,
|
func(c *models.ChannelEdgeInfo,
|
||||||
|
|||||||
Reference in new issue
Block a user