Skip to content

Commit 73ed5a1

Browse files
authored
chore(logs): log url stats as trace (#30421)
1 parent 562c537 commit 73ed5a1

File tree

2 files changed

+21
-15
lines changed

2 files changed

+21
-15
lines changed

lib/util/stats.spec.ts

Lines changed: 19 additions & 14 deletions
Original file line numberDiff line numberDiff line change
@@ -380,7 +380,7 @@ describe('util/stats', () => {
380380

381381
HttpStats.report();
382382

383-
expect(logger.logger.trace).toHaveBeenCalledTimes(1);
383+
expect(logger.logger.trace).toHaveBeenCalledTimes(2);
384384
const [traceData, traceMsg] = logger.logger.trace.mock.calls[0];
385385
expect(traceMsg).toBe('HTTP full statistics');
386386
expect(traceData).toEqual({
@@ -424,6 +424,24 @@ describe('util/stats', () => {
424424
},
425425
});
426426

427+
const [trace2Data, trace2Msg] = logger.logger.trace.mock.calls[1];
428+
expect(trace2Msg).toBe('HTTP URL statistics');
429+
expect(trace2Data).toEqual({
430+
urls: {
431+
'https://example.com/bar': {
432+
GET: {
433+
'200': 1,
434+
},
435+
},
436+
'https://example.com/foo': {
437+
GET: {
438+
'200': 2,
439+
'404': 1,
440+
},
441+
},
442+
},
443+
});
444+
427445
expect(logger.logger.debug).toHaveBeenCalledTimes(1);
428446
const [debugData, debugMsg] = logger.logger.debug.mock.calls[0];
429447
expect(debugMsg).toBe('HTTP statistics');
@@ -440,19 +458,6 @@ describe('util/stats', () => {
440458
},
441459
},
442460
requests: 4,
443-
urls: {
444-
'https://example.com/bar': {
445-
GET: {
446-
'200': 1,
447-
},
448-
},
449-
'https://example.com/foo': {
450-
GET: {
451-
'200': 2,
452-
'404': 1,
453-
},
454-
},
455-
},
456461
});
457462
});
458463
});

lib/util/stats.ts

Lines changed: 2 additions & 1 deletion
Original file line numberDiff line numberDiff line change
@@ -232,7 +232,8 @@ export class HttpStats {
232232
const { urls, rawRequests, hostRequests, hosts, requests } =
233233
HttpStats.getReport();
234234
logger.trace({ rawRequests, hostRequests }, 'HTTP full statistics');
235-
logger.debug({ urls, hosts, requests }, 'HTTP statistics');
235+
logger.debug({ hosts, requests }, 'HTTP statistics');
236+
logger.trace({ urls }, 'HTTP URL statistics');
236237
}
237238
}
238239

0 commit comments

Comments
 (0)