Skip to content

Commit 757c15b

Browse files
committed
CLDSRV-750: skip server access logs for heatbeat and backbeat routes
1 parent 60e7720 commit 757c15b

File tree

1 file changed

+16
-12
lines changed

1 file changed

+16
-12
lines changed

lib/server.js

Lines changed: 16 additions & 12 deletions
Original file line numberDiff line numberDiff line change
@@ -123,20 +123,24 @@ class S3Server {
123123
monitoringClient.httpActiveRequests.inc();
124124
const requestStartTime = process.hrtime.bigint();
125125

126-
// eslint-disable-next-line no-param-reassign
127-
req.serverAccessLog = {
128-
enabled: false,
129-
startTime: requestStartTime,
130-
};
131-
// eslint-disable-next-line no-param-reassign
132-
res.serverAccessLog = {};
126+
// Skip server access logs for heartbeat and backbeat.
127+
if (!req.url.startsWith('/_/')) {
128+
// eslint-disable-next-line no-param-reassign
129+
req.serverAccessLog = {
130+
enabled: false,
131+
startTime: requestStartTime,
132+
};
133133

134-
res.on('finish', () => {
135134
// eslint-disable-next-line no-param-reassign
136-
req.serverAccessLog.endTime = process.hrtime.bigint();
137-
logServerAccess(req.serverAccessLog, req, res);
138-
});
139-
135+
res.serverAccessLog = {};
136+
137+
res.on('finish', () => {
138+
// eslint-disable-next-line no-param-reassign
139+
req.serverAccessLog.endTime = process.hrtime.bigint();
140+
logServerAccess(req.serverAccessLog, req, res);
141+
});
142+
}
143+
140144
// disable nagle algorithm
141145
req.socket.setNoDelay();
142146
res.on('close', () => {

0 commit comments

Comments
 (0)