From 60e12eff996c2665215e4dee8f7a88d0863a70a2 Mon Sep 17 00:00:00 2001 From: Konrad Dzwinel Date: Tue, 26 Jan 2021 22:59:41 +0100 Subject: [PATCH 1/2] Retrieve data from all storage mechanisms (8 -> 14). --- privacy-protections/storage-blocking/main.js | 2 +- 1 file changed, 1 insertion(+), 1 deletion(-) diff --git a/privacy-protections/storage-blocking/main.js b/privacy-protections/storage-blocking/main.js index 1107f49..87ce9fa 100644 --- a/privacy-protections/storage-blocking/main.js +++ b/privacy-protections/storage-blocking/main.js @@ -249,7 +249,7 @@ function retrieveData() { testsSummaryDiv.innerText = `Retrieved data from ${all} storage mechanisms${failed > 0 ? ` (${failed} failed)` : ''}. Click for details.`; } - tests.forEach(test => { + tests.concat(commonTests).forEach(test => { all++; const resultObj = { From eb1a1a52998ccd5047172dadf088a2ecfdb4aa17 Mon Sep 17 00:00:00 2001 From: Konrad Dzwinel Date: Tue, 26 Jan 2021 23:45:29 +0100 Subject: [PATCH 2/2] https test page - Instead of reporting the url that server sees return url that the browser requests --- privacy-protections/https-upgrades/main.js | 3 +-- server.js | 1 + 2 files changed, 2 insertions(+), 2 deletions(-) diff --git a/privacy-protections/https-upgrades/main.js b/privacy-protections/https-upgrades/main.js index d001112..d505236 100644 --- a/privacy-protections/https-upgrades/main.js +++ b/privacy-protections/https-upgrades/main.js @@ -68,8 +68,7 @@ const tests = [ id: 'upgrade-subrequest', run: () => { return fetch(`http://${TEST_DOMAIN}/reflect-headers`) - .then(r => r.json()) - .then(data => data.url); + .then(r => r.url); } }, { diff --git a/server.js b/server.js index f8e81c9..55950d7 100644 --- a/server.js +++ b/server.js @@ -9,6 +9,7 @@ const fs = require('fs'); function fullUrl(req) { return url.format({ + // note: if server is behind a proxy, and it probably is, you may see 'http' here even if request was 'https' protocol: req.protocol, host: req.get('host'), pathname: req.originalUrl