[ajaysehwal]

The docs for route_parameters say the given values override the defaults per field. max_total_routing_fee_msat is an Option where None means "use the default" (1% of the amount plus 50 sats). LDK honors this for BOLT11 and BOLT12 payments, but spontaneous payments and BOLT11 probes copied the field unconditionally. Passing route parameters without a fee limit therefore replaced the default with None, meaning no limit at all. This was found via ldk-server, whose CLI always sends route parameters for spontaneous-send (lightningdevkit/ldk-server#326).

This moves the logic into one helper that only overrides the fee limit when the config sets one, and uses it in SpontaneousPayment::send_inner, Bolt11Payment::send_probes and send_probes_using_amount. Other fields are applied as before, and an explicit limit, including 0, still overrides the default.

Notes:

• BOLT12 async static invoices aren't covered. In rust-lightning, send_payment_for_static_invoice assigns the fee limit unconditionally, so that needs an LDK fix.
• The node-wide Config::route_parameters uses the same helper, so a config without a fee limit now keeps the default too.

Tests: new unit tests in config.rs cover the unset case (the default is kept and the other fields are applied) and explicit limits (0, 1,234, u64::MAX). The first test fails on the old behavior. cargo test --lib passes, and I didn't run the integration tests.

[ldk-reviews-bot]

👋 Thanks for assigning @tnull as a reviewer!
I'll wait for their review and will help manage the review process.
Once they submit their review, I'll check if a second reviewer would be helpful.

[tnull]

Thanks! Looks good, but let's drop the tests as we don't need to cover that simple of behavior (and the tests can't assert uniformity of behavior anyways)