Skip to content

Commit 786d7d1

Browse files
fix: use api key in all requests (ONCALL-7076) (#136)
* use api key in all requests * prevent leaking request details * fix * fix
1 parent 0cf400d commit 786d7d1

File tree

2 files changed

+3
-3
lines changed

2 files changed

+3
-3
lines changed

routes/wpt.js

Lines changed: 1 addition & 1 deletion
Original file line numberDiff line numberDiff line change
@@ -37,7 +37,7 @@ const routeCallback = (error, result, res, rollBarMsg) => {
3737
if (error.statusCode) {
3838
res.status(error.statusCode).send();
3939
} else {
40-
res.json(error);
40+
res.json({status: error.status, message: error.message});
4141
}
4242
} else {
4343
res.json(result);

wtp/apiCaller.js

Lines changed: 2 additions & 2 deletions
Original file line numberDiff line numberDiff line change
@@ -23,7 +23,7 @@ const getTestResults = async (testId, quality, cb) => {
2323
method: "GET",
2424
url: RESULTS_URL,
2525
searchParams: {test: testId},
26-
headers: { 'User-Agent': 'WebSpeedTest' }
26+
headers: { 'User-Agent': 'WebSpeedTest', 'X-WPT-API-KEY': apiKeys.getRandom() }
2727
};
2828
let response;
2929
let rollBarMsg = {};
@@ -117,7 +117,7 @@ const checkTestStatus = async (testId, quality, cb) => {
117117
method: "GET",
118118
url: GET_TEST_STATUS,
119119
searchParams: {test: testId, f: "json"},
120-
'headers': { 'User-Agent': 'WebSpeedTest' }
120+
'headers': { 'User-Agent': 'WebSpeedTest', 'X-WPT-API-KEY': apiKeys.getRandom() }
121121
};
122122
let response;
123123
let rollBarMsg = {};

0 commit comments

Comments
 (0)