diff --git a/cmd/lncli/commands.go b/cmd/lncli/commands.go index c555e584..9b8e6429 100644 --- a/cmd/lncli/commands.go +++ b/cmd/lncli/commands.go @@ -3515,8 +3515,9 @@ var forwardingHistoryCommand = cli.Command{ Query the HTLC switch's internal forwarding log for all completed payment circuits (HTLCs) over a particular time range (--start_time and --end_time). The start and end times are meant to be expressed in - seconds since the Unix epoch. If a start and end time aren't provided, - then events over the past 24 hours are queried for. + seconds since the Unix epoch. If --start_time isn't provided, + then 24 hours ago is used. If --end_time isn't provided, + then the current time is used. The max number of events returned is 50k. The default number is 100, callers can use the --max_events param to modify this value. diff --git a/rpcserver.go b/rpcserver.go index c2125daa..560007c5 100644 --- a/rpcserver.go +++ b/rpcserver.go @@ -4610,14 +4610,19 @@ func (r *rpcServer) ForwardingHistory(ctx context.Context, numEvents uint32 ) - // If the start and end time were not set, then we'll just return the - // records over the past 24 hours. - if req.StartTime == 0 && req.EndTime == 0 { + // If the start time wasn't specified, we'll default to 24 hours ago. + if req.StartTime == 0 { now := time.Now() startTime = now.Add(-time.Hour * 24) - endTime = now } else { startTime = time.Unix(int64(req.StartTime), 0) + } + + // If the end time wasn't specified, assume a default end time of now. + if req.EndTime == 0 { + now := time.Now() + endTime = now + } else { endTime = time.Unix(int64(req.EndTime), 0) } @@ -4628,7 +4633,7 @@ func (r *rpcServer) ForwardingHistory(ctx context.Context, numEvents = 100 } - // Next, we'll map the proto request into a format the is understood by + // Next, we'll map the proto request into a format that is understood by // the forwarding log. eventQuery := channeldb.ForwardingEventQuery{ StartTime: startTime,