lightningdevkit / rust-lightning

A highly modular Bitcoin Lightning library written in Rust. It's rust-lightning, not Rusty's Lightning!
Other
1.16k stars 367 forks source link

Replace verbose event processing logging #3331

Open tnull opened 1 month ago

tnull commented 1 month ago

In https://github.com/lightningdevkit/rust-lightning/pull/3314 we introduced logging around event processing.

Logging in the event processing loop will however turn out to be very spammy, in particular since we recently introduced fallible event handling in which case we directly loop around and retry processing.

We should move the logging to calling handle_event (see https://github.com/lightningdevkit/rust-lightning/pull/3314#issuecomment-2353006381) but probably want to introduce some kind of delay before retrying event handling (which we discussed before, but didn't actually do in #2995.