Skip to content

Commit

Permalink
Restore ForwardFeeMonitor's ability to see forwarding fees
Browse files Browse the repository at this point in the history
Fixes ([#222])

Prior to ElementsProject/lightning@780f32d (`v23.05`) both `fee` and
`fee_msat` were sent for compatibility.  The ForwardFeeMonitor was
checking for the presence of the `fee` field before processing the
record.  This needed to be updated to `fee_msat`.
  • Loading branch information
ksedgwic committed Aug 8, 2024
1 parent 29fad23 commit a36d119
Show file tree
Hide file tree
Showing 2 changed files with 1 addition and 2 deletions.
2 changes: 1 addition & 1 deletion Boss/Mod/ForwardFeeMonitor.cpp
Original file line number Diff line number Diff line change
Expand Up @@ -29,7 +29,7 @@ void ForwardFeeMonitor::start() {
try {
auto payload = n.params["forward_event"];
if ( !payload.has("out_channel")
|| !payload.has("fee")
|| !payload.has("fee_msat")
|| !payload.has("resolved_time")
|| !payload.has("received_time")
)
Expand Down
1 change: 0 additions & 1 deletion tests/boss/test_forwardfeemonitor.cpp
Original file line number Diff line number Diff line change
Expand Up @@ -131,7 +131,6 @@ int main() {
"in_msat": "100001001msat",
"out_msatoshi": 100000000,
"out_msat": "100000000msat",
"fee": 1001,
"fee_msat": "1001msat",
"status": "settled",
"received_time": 1560696342.368,
Expand Down

0 comments on commit a36d119

Please sign in to comment.