Minor While I was adding some more tests to detect error logs coming from Node in the container, I stumbled into the following one:
info: Creating registration..
error: Failed to register: Unavailable: request to http://backend:3030/registration failed, reason: connect ECONNREFUSED 172.17.0.3:3030
at new Unavailable (/evergreen/client/node_modules/@feathersjs/errors/lib/index.js:191:17)
at toError (/evergreen/client/node_modules/@feathersjs/rest-client/lib/base.js:9:11)
at <anonymous>
at process._tickCallback (internal/process/next_tick.js:188:7)
(node:32) UnhandledPromiseRejectionWarning: Unhandled promise rejection (rejection id: 2): Unavailable: request to http://backend:3030/registration failed, reason: connect ECONNREFUSED 172.17.0.3:3030
(node:32) [DEP0018] DeprecationWarning: Unhandled promise rejections are deprecated. In the future, promise rejections that are not handled will terminate the Node.js process with a non-zero exit code.
So I'm going to add a workaround to the test for now, and maybe this is already fixed in the ongoing upgrade-level PR, but I thought I'd file this to make sure we do not forget.
- links to