Archived
fix: omit readiness request logs
This commit is contained in:
@@ -23,6 +23,7 @@ export function observeHttpRequest(
|
|||||||
response: ServerResponse,
|
response: ServerResponse,
|
||||||
write: (entry: HttpRequestLog) => void,
|
write: (entry: HttpRequestLog) => void,
|
||||||
): void {
|
): void {
|
||||||
|
if (request.url === "/readyz") return;
|
||||||
const startedAt = Date.now();
|
const startedAt = Date.now();
|
||||||
let observed = false;
|
let observed = false;
|
||||||
const observe = () => {
|
const observe = () => {
|
||||||
|
|||||||
@@ -119,6 +119,12 @@ test("invalid webhook signatures are rejected without persistence", async () =>
|
|||||||
},
|
},
|
||||||
);
|
);
|
||||||
assert.ok((requests[0]?.durationMs ?? -1) >= 0);
|
assert.ok((requests[0]?.durationMs ?? -1) >= 0);
|
||||||
|
|
||||||
|
const readiness = await fetch(
|
||||||
|
`http://127.0.0.1:${address.port}/readyz`,
|
||||||
|
);
|
||||||
|
assert.equal(readiness.status, 200);
|
||||||
|
assert.equal(requests.length, 1);
|
||||||
} finally {
|
} finally {
|
||||||
await new Promise<void>((resolve) => server.close(() => resolve()));
|
await new Promise<void>((resolve) => server.close(() => resolve()));
|
||||||
store.close();
|
store.close();
|
||||||
|
|||||||
Reference in New Issue
Block a user