specklesystems / speckle-server

The Speckle Server, Frontend, 3D Viewer, & other JS utilities.
https://speckle.systems
Other
667 stars 168 forks source link

fix(logging): log errors thrown before request context is present #3165

Open iainsproat opened 3 days ago

iainsproat commented 3 days ago

Description & motivation

The default error handler assumed that the request context would be present. Where errors are thrown by Express middleware, the request context was not yet present resulting in errors not being properly logged.

Changes:

To-do before merge:

Screenshots:

Validation of changes:

Screenshot 2024-10-01 at 12 16 55

Checklist:

References