mirror of
https://github.com/Ride-The-Lightning/RTL.git
synced 2026-08-20 13:27:56 +02:00
4ee6817906
Follow-up to the second #1629 review: - F4: the limiter invokes its done callback outside the surrounding .then/.catch, so a throw in the response-send body became an unhandled rejection with no response (a 500 -> hang regression, notably on LND postPeer where the inner .catch was removed). Wrap each converted done body in try/catch that sends the error response, guarded by res.headersSent. - F5: CLN postPeer re-listed peers but never resolved their aliases, so a freshly connected CLN peer came back with a raw node id (the frontend uses this response directly). Resolve aliases through the same bounded limiter, matching LND postPeer. - F6: make runWithConcurrencyLimit fire 'done' exactly once via a one-shot guard, so multiple synchronous completions (e.g. non-function task elements) can't double-send the response. |
||
|---|---|---|
| .. | ||
| balance.ts | ||
| channels.ts | ||
| channelsBackup.ts | ||
| fees.ts | ||
| getInfo.ts | ||
| graph.ts | ||
| invoices.ts | ||
| message.ts | ||
| newAddress.ts | ||
| payments.ts | ||
| peers.ts | ||
| switch.ts | ||
| transactions.ts | ||
| wallet.ts | ||
| webSocketClient.ts | ||