Add content-negotiation

This commit is contained in:
Timothy Warren 2016-02-19 16:23:00 -05:00
parent b6dc717a00
commit a57b86257c
5 changed files with 33 additions and 11 deletions

View File

@ -2,6 +2,7 @@
const errors = require('errors'); const errors = require('errors');
const express = require('express'); const express = require('express');
const negotiate = require('express-negotiate');
const path = require('path'); const path = require('path');
const autoLoad = require('./config/container-autoload'); const autoLoad = require('./config/container-autoload');

View File

@ -8,15 +8,21 @@ const container = require('../Container');
const app = container.get('app'); const app = container.get('app');
const HTTP_CODE_MAP = require('http').STATUS_CODES; const HTTP_CODE_MAP = require('http').STATUS_CODES;
const errors = require('errors'); const errors = require('errors');
const negotiate = require('express-negotiate');
let errorHandlers = new Set([ let errorHandlers = new Set([
function handle404(req, res, next) { function handle400Errors(err, req, res, next) {
// if no route matches, send a 404
if (! req.route) { if (! req.route) {
let err = new errors.Http404Error(); // if no route matches, send a 404
return next(err); err = new errors.Http404Error();
} else if (err instanceof negotiate.NotAcceptable) {
// if no content type matches, send a 406
err = new errors.Http406Error();
} }
return next(err);
}, },
// general error handler // general error handler
@ -36,12 +42,19 @@ let errorHandlers = new Set([
output.error = err; output.error = err;
} }
res.render('error', { // Send html or json depending on client accept header
title: `${err.status} ${err.message}`, req.negotiate({
error: output, html: () => {
}); res.render('error', {
title: `${err.status} ${err.message}`,
error: output,
});
},
//res.json(output); 'application/json': () => {
res.json(output);
},
});
}, },
]); ]);

View File

@ -4,8 +4,12 @@ module.exports = {
'/': { '/': {
// Get homepage // Get homepage
get: (req, res) => { get: (req, res) => {
return res.render('index', { req.negotiate({
title: 'Blog test page', html: () => {
return res.render('index', {
title: 'Blog test page',
});
},
}); });
}, },
}, },

View File

@ -3,14 +3,17 @@
<h2>{{error.message}}</h2> <h2>{{error.message}}</h2>
{{#if error.error}} {{#if error.error}}
<article> <article>
<center>
<table> <table>
<caption>Error Details</caption> <caption>Error Details</caption>
{{#each error as |value key|}} {{#each error as |value key|}}
<tr> <tr>
<td>{{key}}</td> <td>{{key}}</td>
<td>{{value}}</td> <td>{{value}}</td>
</tr>
{{/each}} {{/each}}
</table> </table>
</center>
</article> </article>
{{/if}} {{/if}}
</main> </main>

View File

@ -11,6 +11,7 @@
"eslint": "^1.10.3", "eslint": "^1.10.3",
"express": "4.*", "express": "4.*",
"express-handlebars": "^3.0.0", "express-handlebars": "^3.0.0",
"express-negotiate": "0.0.5",
"express-session": "^1.13.0", "express-session": "^1.13.0",
"getargs": "0.0.8", "getargs": "0.0.8",
"glob": "^6.0.4", "glob": "^6.0.4",