illinois / queue

A microservice queue for holding open office hours
University of Illinois/NCSA Open Source License
82 stars 36 forks source link

Add debugging for new socket error #258

Closed nwalters512 closed 5 years ago

nwalters512 commented 5 years ago

We're now seeing a fun new variety of error relating to setting up sockets:

Unhandled rejection (promise:  Promise {
 <rejected> TypeError: Cannot destructure property `courseId` of 'undefined' or 'null'.
   at Socket.socket.on (/home/waf/queue/build/socket/server.js:233:46)
   at <anonymous> } , reason:  TypeError: Cannot destructure property `courseId` of 'undefined' or 'null'.
   at Socket.socket.on (/home/waf/queue/build/socket/server.js:233:46)
   at <anonymous> ).

This PR adds explicit logging for when that fails to hopefully help trace the problem back to its source.

vercel[bot] commented 5 years ago

This pull request is automatically deployed with Now. To access deployments, click Details below or on the icon next to each push.