question: HttpError returned as string instead of JSON

System:

node: v12.9.1 npm: 6.10.2 typescript: 3.7.5

index.ts

import 'reflect-metadata';
import { createExpressServer } from 'routing-controllers';

const app = createExpressServer({
  controllers: [__dirname + '/**/*.controller.js'], // even if reflection isn't used and the Controller is imported directly it doesn't work
});
app.listen(3000);

example/example.controller.ts

import { Controller, NotFoundError, Get } from "routing-controllers";

@Controller('/example')
export class ExampleController {
  @Get()
  public get(): string {
    throw new NotFoundError('what!');
  }
}

Causes the following error to be produced

Error at NotFoundError.HttpError [as constructor] (/Users/john.doe/Documents/code/routing-controllers/node_modules/routing-controllers/http-error/HttpError.js:30:23) at new NotFoundError (/Users/john.doe/Documents/code/routing-controllers/node_modules/routing-controllers/http-error/NotFoundError.js:23:28) at ExampleController.get (/Users/john.doe/Documents/code/routing-controllers/dist/example/example.controller.js:17:15) at ActionMetadata.callMethod (/Users/john.doe/Documents/code/routing-controllers/node_modules/routing-controllers/metadata/ActionMetadata.js:108:48) at /Users/john.doe/Documents/code/routing-controllers/node_modules/routing-controllers/RoutingControllers.js:99:140 at processTicksAndRejections (internal/process/task_queues.js:85:5)

According to the documentation I should expect

{
  "name": "NotFoundError",
  "message": "what!"
}

[jotamorais]

@dioio-io, @ahmedabdelhafez, @marines, @franckbattu this is expected behavior when NODE_ENV is not production.

(property) RoutingControllersOptions.development?: boolean Indicates if development mode is enabled. By default, it’s enabled if your NODE_ENV is not equal to “production”.

To test locally, you can set the development property of RoutingControllersOptions, to ‘false’.

import "reflect-metadata";
import { createExpressServer } from "routing-controllers";

createExpressServer({
  development: false,
  controllers: [__dirname + "/**/*.controller.ts"] // even if reflection isn't used and the Controller is imported directly it doesn't work
}).listen(3000);

I’ve created this sandbox to show.

Similar: #473, #460, #440,

[NoNameProvided]

This issue has been automatically locked since there has not been any recent activity after it was closed. Please open a new issue for related bugs.