Possible bug or spec? about error-tracker

[jridgewell]

Hi @ppashakhanloo: Sanitization happens in

error-tracker/utils/requests/extract-reporting-params.js

Line 23 in 224fe25

function extractReportingParams(params) {

and

error-tracker/routes/error-tracker.js

Lines 109 to 124 in 224fe25

	if (!referrer || !version || !message) {
	return res.sendStatus(statusCodes.BAD_REQUEST);
	}
	// Accept but ignore requests that get throttled.
	if (
	version.includes('internalRuntimeVersion') ||
	Math.random() > logTarget.throttleRate
	) {
	return res.sendStatus(statusCodes.OK);
	}
	const rtvs = await latestRtv();
	// Drop requests from RTVs that are no longer being served.
	if (rtvs.length > 0 && !rtvs.includes(version)) {
	return res.sendStatus(statusCodes.GONE);
	}

.

The deleter call itself shouldn't matter for the param body, it will only delete cache entries that we generate to ensure the cache doesn't grow unbounded.