xiaoxue_admin
/
RSSHub
1
0
Fork 0
Code Issues Pull Requests Packages Projects Releases Wiki Activity

fix: only collect errors which route path was matched (#4673)

This commit is contained in:
howel52 2020-05-08 11:27:48 +08:00 committed by GitHub
parent 283468d43a
commit 3e846e6919
No known key found for this signature in database
GPG Key ID: 4AEE18F83AFDEB23
3 changed files with 8 additions and 8 deletions
Show Stats Download Patch File Download Diff File Expand all files Collapse all files

4
lib/middleware/debug.js
View File

@ -14,9 +14,9 @@ module.exports = async (ctx, next) => {
await next();
if (!ctx.debug.routes[ctx._matchedRoute]) {
ctx.debug.routes[ctx._matchedRoute] = 0;
ctx._matchedRoute && (ctx.debug.routes[ctx._matchedRoute] = 0);
}
ctx.debug.routes[ctx._matchedRoute]++;
ctx._matchedRoute && ctx.debug.routes[ctx._matchedRoute]++;
if (ctx.response.get('X-Koa-Redis-Cache') || ctx.response.get('X-Koa-Memory-Cache')) {
ctx.debug.hitCache++;

8
lib/middleware/onerror.js
View File

@ -51,15 +51,15 @@ module.exports = async (ctx, next) => {
ctx.debug.errorPaths[ctx.request.path]++;
if (!ctx.debug.errorRoutes[ctx._matchedRoute]) {
ctx.debug.errorRoutes[ctx._matchedRoute] = 0;
ctx._matchedRoute && (ctx.debug.errorRoutes[ctx._matchedRoute] = 0);
}
ctx.debug.errorRoutes[ctx._matchedRoute]++;
ctx._matchedRoute && ctx.debug.errorRoutes[ctx._matchedRoute]++;
if (!ctx.state.debuged) {
if (!ctx.debug.routes[ctx._matchedRoute]) {
ctx.debug.routes[ctx._matchedRoute] = 0;
ctx._matchedRoute && (ctx.debug.routes[ctx._matchedRoute] = 0);
}
ctx.debug.routes[ctx._matchedRoute]++;
ctx._matchedRoute && ctx.debug.routes[ctx._matchedRoute]++;
if (ctx.response.get('X-Koa-Redis-Cache') || ctx.response.get('X-Koa-Memory-Cache')) {
ctx.debug.hitCache++;

4
test/middleware/debug.js
View File

@ -43,7 +43,7 @@ describe('debug', () => {
expect(value).toBe('8');
break;
case 'hot routes:':
expect(value).toBe('4 undefined<br>3 /test/:id<br>');
expect(value).toBe('3 /test/:id<br>');
break;
case 'hot paths:':
expect(value).toBe('3 /test/1<br>2 /test/2<br>2 /test/empty<br>1 /<br>');
@ -52,7 +52,7 @@ describe('debug', () => {
expect(value).toBe('5 233.233.233.233<br>3 233.233.233.234<br>');
break;
case 'hot error routes:':
expect(value).toBe('1 /test/:id<br>1 undefined<br>');
expect(value).toBe('1 /test/:id<br>');
break;
case 'hot error paths:':
expect(value).toBe('2 /test/empty<br>');