plugin: Always set the completion time, independently of outcome

Reported-by: @thestick613 Fixes #3848
ElementsProject · Jul 18, 2020 · c75b0ac · c75b0ac
1 parent 65ca634
commit c75b0ac
Showing 1 changed file with 1 addition and 1 deletion.
diff --git a/plugins/libplugin-pay.c b/plugins/libplugin-pay.c
@@ -661,6 +661,7 @@ payment_waitsendpay_finished(struct command *cmd, const char *buffer,
 	struct route_hop *hop;
 	assert(p->route != NULL);
 
+	p->end_time = time_now();
 	p->result = tal_sendpay_result_from_json(p, buffer, toks);
 
 	if (p->result == NULL) {
@@ -677,7 +678,6 @@ payment_waitsendpay_finished(struct command *cmd, const char *buffer,
 
 	if (p->result->state == PAYMENT_COMPLETE) {
 		payment_set_step(p, PAYMENT_STEP_SUCCESS);
-		p->end_time = time_now();
 		payment_continue(p);
 		return command_still_pending(cmd);
 	}