From 29bc655909a5c3a2a9c13df91a3fe382221583be Mon Sep 17 00:00:00 2001 From: Jakob Ackermann Date: Wed, 30 Nov 2022 14:36:56 +0000 Subject: [PATCH] Merge pull request #10725 from overleaf/jpa-no-cdn-metric [web] add metric for ?nocdn=true requests GitOrigin-RevId: 6ca8a5616590f2d4efd8e3d04ef3ae71b67c5b96 --- services/web/app/src/infrastructure/ExpressLocals.js | 5 +++++ 1 file changed, 5 insertions(+) diff --git a/services/web/app/src/infrastructure/ExpressLocals.js b/services/web/app/src/infrastructure/ExpressLocals.js index 84cda0d349..6209ed51af 100644 --- a/services/web/app/src/infrastructure/ExpressLocals.js +++ b/services/web/app/src/infrastructure/ExpressLocals.js @@ -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