Swallowing errors about magic HOT 3 CLOSED

swagger-tools' initializeMiddleware was not ever intended to be an "error-first callback" and is not documented to be. There is no safe way to handle an error so passing it back to the caller makes no sense. The code in swagger-tools is doing the right thing. This issue is caused by the version of json-refs in swagger-tools being broken so upgrading to a version that fixes it should fix this issue.

I've now had to reply with a similar response to three separate issues filed in three separate projects. Fixing this in [email protected] means magic will get the fix automatically and itself does not require any changes to fix this. I will let @theganyo decide whether or not to close this as it's already being addressed in swagger-tools.

from magic.

Then can someone explain how I'm supposed to figure out why my Node.js application is crashing when these modules simply hide the error message which would immediately give me an answer on your platform?

To reproduce from the very README on this project:

var a127 = require('a127-magic');
var app = require('express')();

a127.init(function(config) {
  throw new Error('You should see this, but you do not');
  app.use(a127.middleware(config));
  app.listen(process.env.PORT || 10010);
});

Process exits, no error, nothing. How is this a feature?

from magic.

I think I've figured it out. I'm closing this and we'll discuss at swagger-tools as this is a swagger-tools issue.

from magic.