commit 747ff4fee1e67b4babd34a4c9282f5f4031223eb
parent 82a139a8f0edddec8ef2e8456c12c7270ca671ab
Author: iulian moraru <imoraru@mozilla.com>
Date: Fri, 21 Nov 2025 02:39:19 +0200
Revert "Bug 2001400 - Add pref for dap.hpke and pass as options in attribution conversion r=home-newtab-reviewers,mconley" for causing xpcshell failures on test_NewTabAttributionService.js.
This reverts commit c80d8f741871a23c36b62d86d604669abdc7cc0e.
Diffstat:
2 files changed, 2 insertions(+), 28 deletions(-)
diff --git a/browser/app/profile/firefox.js b/browser/app/profile/firefox.js
@@ -676,12 +676,6 @@ pref("browser.urlbar.merino.ohttpConfigURL", "");
// OHTTP relay URL for Merino requests
pref("browser.urlbar.merino.ohttpRelayURL", "");
-// OHTTP hpke for DAP
-pref("dap.ohttp.hpke", "https://dap-09-3.api.divviup.org/ohttp-configs");
-
-// OHTTP relay URL for DAP
-pref("dap.ohttp.relayURL", "https://mozilla-ohttp-dap.mozilla.fastly-edge.com/");
-
// Enable site specific search result.
pref("browser.urlbar.contextualSearch.enabled", true);
diff --git a/browser/extensions/newtab/lib/NewTabAttributionService.sys.mjs b/browser/extensions/newtab/lib/NewTabAttributionService.sys.mjs
@@ -15,9 +15,6 @@ const MAX_LOOKBACK_DAYS = 30;
const DAY_IN_MILLI = 1000 * 60 * 60 * 24;
const CONVERSION_RESET_MILLI = 7 * DAY_IN_MILLI;
-const DAP_HPKE_PREF = "dap.ohttp.hpke";
-const DAP_RELAY_PREF = "dap.ohttp.relayURL";
-
/**
*
*/
@@ -178,14 +175,7 @@ class NewTabAttributionService {
if (lookbackDays > MAX_LOOKBACK_DAYS) {
return;
}
- const dapHpke = Services.prefs.getCharPref(
- DAP_HPKE_PREF,
- "https://dap-09-3.api.divviup.org/ohttp-configs"
- );
- const ohttpRelayURL = Services.prefs.getCharPref(
- DAP_RELAY_PREF,
- "https://mozilla-ohttp-dap.mozilla.fastly-edge.com/"
- );
+
const now = this.#now();
const budget = await this.#getBudget(partnerId, now);
@@ -215,20 +205,10 @@ class NewTabAttributionService {
}
await this.#updateBudget(budget, budgetSpend, partnerId);
-
- const options = {};
- if (dapHpke) {
- options.ohttp_hpke = dapHpke;
- }
-
- if (ohttpRelayURL) {
- options.ohttp_relay = ohttpRelayURL;
- }
-
await this.#dapSender.sendDAPMeasurement(
conversion.task,
measurement,
- options
+ {}
);
} catch (e) {
console.error(e);