RTL/server
saubyk e0fce065d5 Address 2nd review: guard limiter callbacks, CLN postPeer aliases, one-shot done
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.
2026-07-19 22:01:23 -07:00
..
controllers Address 2nd review: guard limiter callbacks, CLN postPeer aliases, one-shot done 2026-07-19 22:01:23 -07:00
models Add Disable Authentication option in the backend 2026-07-19 22:01:23 -07:00
routes Update project code according to dependencies update 2026-02-01 09:06:07 -08:00
utils Address 2nd review: guard limiter callbacks, CLN postPeer aliases, one-shot done 2026-07-19 22:01:23 -07:00
tsconfig.server.json Update project code according to dependencies update 2026-02-01 09:06:07 -08:00