From e56b7ead3ebbc77e415913e0d26e47f5ae573775 Mon Sep 17 00:00:00 2001 From: =?UTF-8?q?=C5=81ukasz=20Czerpak?= Date: Thu, 5 Jun 2025 18:22:17 +0200 Subject: [PATCH 1/4] use api key in all requests --- wtp/apiCaller.js | 4 ++-- 1 file changed, 2 insertions(+), 2 deletions(-) diff --git a/wtp/apiCaller.js b/wtp/apiCaller.js index e8aa8b2..cfc1778 100644 --- a/wtp/apiCaller.js +++ b/wtp/apiCaller.js @@ -23,7 +23,7 @@ const getTestResults = async (testId, quality, cb) => { method: "GET", url: RESULTS_URL, searchParams: {test: testId}, - headers: { 'User-Agent': 'WebSpeedTest' } + headers: { 'User-Agent': 'WebSpeedTest', 'X-WPT-API-KEY': apiKeys.getRandom() } }; let response; let rollBarMsg = {}; @@ -117,7 +117,7 @@ const checkTestStatus = async (testId, quality, cb) => { method: "GET", url: GET_TEST_STATUS, searchParams: {test: testId, f: "json"}, - 'headers': { 'User-Agent': 'WebSpeedTest' } + 'headers': { 'User-Agent': 'WebSpeedTest', 'X-WPT-API-KEY': apiKeys.getRandom() } }; let response; let rollBarMsg = {}; From 6cf0bd9419ac2c03c4a1eadae9872bbc88aeaf0a Mon Sep 17 00:00:00 2001 From: =?UTF-8?q?=C5=81ukasz=20Czerpak?= Date: Thu, 5 Jun 2025 18:50:00 +0200 Subject: [PATCH 2/4] prevent leaking request details --- routes/wpt.js | 2 +- 1 file changed, 1 insertion(+), 1 deletion(-) diff --git a/routes/wpt.js b/routes/wpt.js index 4c1d3a8..15751b5 100644 --- a/routes/wpt.js +++ b/routes/wpt.js @@ -37,7 +37,7 @@ const routeCallback = (error, result, res, rollBarMsg) => { if (error.statusCode) { res.status(error.statusCode).send(); } else { - res.json(error); + res.json({message: error.message}); } } else { res.json(result); From 8f500dd764e1a8decde3fc97e55d3a178eedcdec Mon Sep 17 00:00:00 2001 From: =?UTF-8?q?=C5=81ukasz=20Czerpak?= Date: Thu, 5 Jun 2025 18:57:10 +0200 Subject: [PATCH 3/4] fix --- routes/wpt.js | 2 +- 1 file changed, 1 insertion(+), 1 deletion(-) diff --git a/routes/wpt.js b/routes/wpt.js index 15751b5..d9abb74 100644 --- a/routes/wpt.js +++ b/routes/wpt.js @@ -37,7 +37,7 @@ const routeCallback = (error, result, res, rollBarMsg) => { if (error.statusCode) { res.status(error.statusCode).send(); } else { - res.json({message: error.message}); + res.json({status: error?.status, message: error?.message}); } } else { res.json(result); From c913e1c73506e7b9c6dd369c8813632ccfce069f Mon Sep 17 00:00:00 2001 From: =?UTF-8?q?=C5=81ukasz=20Czerpak?= Date: Thu, 5 Jun 2025 18:59:19 +0200 Subject: [PATCH 4/4] fix --- routes/wpt.js | 2 +- 1 file changed, 1 insertion(+), 1 deletion(-) diff --git a/routes/wpt.js b/routes/wpt.js index d9abb74..085aa0a 100644 --- a/routes/wpt.js +++ b/routes/wpt.js @@ -37,7 +37,7 @@ const routeCallback = (error, result, res, rollBarMsg) => { if (error.statusCode) { res.status(error.statusCode).send(); } else { - res.json({status: error?.status, message: error?.message}); + res.json({status: error.status, message: error.message}); } } else { res.json(result);