self-hosted/joplin
1
0
Fork 0
mirror of https://github.com/laurent22/joplin.git synced 2025-07-10 00:05:42 +02:00
Code Issues Releases Activity

Server: Improved logging and rendering of low level middleware errors

Browse Source
This commit is contained in:
Laurent Cozic
2021-11-01 19:20:36 +00:00
parent c49174124e
commit 370441333f
1 changed files with 23 additions and 11 deletions
Download Patch File Download Diff File Expand all files Collapse all files

34
packages/server/src/app.ts
View File

@ -19,6 +19,7 @@ import apiVersionHandler from './middleware/apiVersionHandler';
import clickJackingHandler from './middleware/clickJackingHandler'; import clickJackingHandler from './middleware/clickJackingHandler';
import newModelFactory from './models/factory'; import newModelFactory from './models/factory';
import setupCommands from './utils/setupCommands'; import setupCommands from './utils/setupCommands';
import { RouteResponseFormat, routeResponseFormat } from './utils/routeUtils';
interface Argv { interface Argv {
env?: Env; env?: Env;
@ -139,17 +140,28 @@ async function main() {
} catch (error) { } catch (error) {
ctx.status = error.httpCode || 500; ctx.status = error.httpCode || 500;
// Since this is a low level error, rendering a view might fail too, appLogger().error(`Middleware error on ${ctx.path}:`, error);
// so catch this and default to rendering JSON.
try { const responseFormat = routeResponseFormat(ctx);
ctx.body = await ctx.joplin.services.mustache.renderView({
name: 'error', if (responseFormat === RouteResponseFormat.Html) {
title: 'Error', // Since this is a low level error, rendering a view might fail too,
path: 'index/error', // so catch this and default to rendering JSON.
content: { error }, try {
}); ctx.response.set('Content-Type', 'text/html');
} catch (anotherError) { ctx.body = await ctx.joplin.services.mustache.renderView({
ctx.body = { error: anotherError.message }; name: 'error',
title: 'Error',
path: 'index/error',
content: { error },
});
} catch (anotherError) {
ctx.response.set('Content-Type', 'application/json');
ctx.body = JSON.stringify({ error: error.message });
}
} else {
ctx.response.set('Content-Type', 'application/json');
ctx.body = JSON.stringify({ error: error.message });
} }
} }
}); });