fix: Use content-type text/plain for errors

This commit is contained in:
Joachim Van Herwegen 2020-07-22 15:21:37 +02:00
parent 0e486cf6a6
commit b93a77c11b
2 changed files with 4 additions and 4 deletions

View File

@ -23,9 +23,8 @@ export class SimpleResponseWriter extends ResponseWriter {
if (input.description) { if (input.description) {
input.response.setHeader('location', input.description.identifier.path); input.response.setHeader('location', input.description.identifier.path);
if (input.description.body) { if (input.description.body) {
if (input.description.body.metadata.contentType) { const contentType = input.description.body.metadata.contentType || 'text/plain';
input.response.setHeader('content-type', input.description.body.metadata.contentType); input.response.setHeader('content-type', contentType);
}
input.description.body.data.pipe(input.response); input.description.body.data.pipe(input.response);
} }
@ -40,6 +39,7 @@ export class SimpleResponseWriter extends ResponseWriter {
if (input.error instanceof HttpError) { if (input.error instanceof HttpError) {
code = input.error.statusCode; code = input.error.statusCode;
} }
input.response.setHeader('content-type', 'text/plain');
input.response.writeHead(code); input.response.writeHead(code);
input.response.end(`${input.error.name}: ${input.error.message}\n${input.error.stack}`); input.response.end(`${input.error.name}: ${input.error.message}\n${input.error.stack}`);
} }

View File

@ -26,7 +26,7 @@ export class ExpressHttpServer {
} catch (error) { } catch (error) {
const errMsg = `${error.name}: ${error.message}\n${error.stack}`; const errMsg = `${error.name}: ${error.message}\n${error.stack}`;
process.stderr.write(errMsg); process.stderr.write(errMsg);
response.status(500).send(errMsg); response.status(500).contentType('text/plain').send(errMsg);
} }
}); });
return app.listen(port); return app.listen(port);