Merge pull request #10725 from overleaf/jpa-no-cdn-metric

[web] add metric for ?nocdn=true requests

GitOrigin-RevId: 6ca8a5616590f2d4efd8e3d04ef3ae71b67c5b96
This commit is contained in:
Jakob Ackermann
2022-12-01 09:05:02 +00:00
committed by Copybot
parent 0ed9626475
commit 648a08b6b8
@@ -1,4 +1,5 @@
const logger = require('@overleaf/logger')
const Metrics = require('@overleaf/metrics')
const Settings = require('@overleaf/settings')
const querystring = require('querystring')
const _ = require('lodash')
@@ -129,6 +130,10 @@ module.exports = function (webRouter, privateApiRouter, publicApiRouter) {
{ user_id: userId, ip: req != null ? req.ip : undefined },
'cdnBlocked for user, not using it and turning it off for future requets'
)
Metrics.inc('no_cdn', 1, {
path: userId ? 'logged-in' : 'pre-login',
method: 'true',
})
req.session.cdnBlocked = true
}
const host = req.headers && req.headers.host