Page MenuHomePhabricator

[lib] Wrap spinPromises in ignorePromiseRejections
ClosedPublic

Authored by ashoat on Dec 4 2023, 1:09 PM.
Tags
None
Referenced Files
Unknown Object (File)
Thu, Oct 3, 9:00 PM
Unknown Object (File)
Sun, Sep 29, 6:35 PM
Unknown Object (File)
Sun, Sep 8, 2:43 AM
Unknown Object (File)
Sat, Sep 7, 9:15 AM
Unknown Object (File)
Sat, Sep 7, 9:15 AM
Unknown Object (File)
Sat, Sep 7, 9:15 AM
Unknown Object (File)
Sat, Sep 7, 9:15 AM
Unknown Object (File)
Aug 31 2024, 4:11 PM
Subscribers

Details

Summary

The unused-promise Flow lint revealed an issue here. We use SequentialPromiseResolver in keyserver, where an unhandled promise rejection could crash the whole server. But we aren't doing anything to handle a potential promise rejection here.

This diff wraps spinPromises in ignorePromiseRejections. This way, if the promise rejects we will print an error instead of crashing the whole keyserver.

Depends on D10160

Test Plan

Flow

Diff Detail

Repository
rCOMM Comm
Lint
Lint Not Applicable
Unit
Tests Not Applicable