Make sendErrorMessage response available on server

Note: for support questions, please use one of these channels: stackoverflow or slack

You want to:

• [] report a bug
• request a feature

Current behaviour

Apologies if this is my misunderstanding of NodeJS streams.

The HTTP body of sendErrorMessage is readable on the client but does not appear to be accessible server side.

I’m currently investigating a number of 400 errors produced here but it looks like this could be 1 of 4 Server.errorMessages.

Are the response bodies available server side or would it be possible to add some debug logging that includes the body before sending the response?

Steps to reproduce (if the current behaviour is a bug)

Expected behaviour

Setup

• OS:
• browser:
• engine.io version:

Other information (e.g. stacktraces, related issues, suggestions how to fix)

[darrachequesne]

Closed by https://github.com/socketio/engine.io/commit/7096e98a02295a62c8ea2aa56461d4875887092d, included in engine.io@5.1.0.

Syntax:

server.on("connection_error", (err) => {
  console.log(err.req);		// the request object
  console.log(err.code);	// the error code, for example 1
  console.log(err.message);	// the error message, for example "Session ID unknown"
  console.log(err.context);     // some additional error context
});

[Khez]

@darrachequesne any chance you can take a look through the PR so we can move this issue forward ?

I’m open to any and all suggestions 😃