mirror of
https://github.com/Ride-The-Lightning/RTL.git
synced 2026-08-13 12:33:07 +02:00
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. |
||
|---|---|---|
| .. | ||
| controllers | ||
| models | ||
| routes | ||
| utils | ||
| tsconfig.server.json | ||