strongloop / loopback-boot

Convention-based bootstrapper for LoopBack applications
Other
62 stars 71 forks source link

fix: set `app.booting` flag immediately #286

Closed bajtos closed 5 years ago

bajtos commented 5 years ago

Before this change, app.booting was set by Application plugin as part of regular phase invocation, which executes individual plugins in subsequent turns of the event loop. As a result, app.booting was initially undefined despite the fact that the booting process was already in progress.

This patch moves manipulation of app.booting flag directly to Bootstrapper#run method, to ensure it's set early enough and always properly cleared at the end.

Related issues

Checklist