Closed
Bug 1915702
Opened 9 months ago
Closed 8 months ago
Update OHTTP Relay URL of shopping
Categories
(Firefox :: Shopping, task)
Firefox
Shopping
Tracking
()
RESOLVED
FIXED
132 Branch
People
(Reporter: tcampbell, Assigned: tcampbell)
References
(Blocks 1 open bug)
Details
Attachments
(2 files)
48 bytes,
text/x-phabricator-request
|
Details | Review | |
48 bytes,
text/x-phabricator-request
|
phab-bot
:
approval-mozilla-beta+
|
Details | Review |
Both https://mozilla-ohttp-fakespot.fastly-edge.com/
and https://mozilla-ohttp.fastly-edge.com/
resolve to the same servers and are interchangeable. For consistency with other Firefox components using OHTTP, let us update the shopping pref to point to the url without fakespot in the name.
Assignee | ||
Comment 1•9 months ago
|
||
For consistency with other uses of OHTTP in Firefox, update the URL. Both old
and new URL are aliases for the same server so behaviours are the same.
Pushed by tcampbell@mozilla.com:
https://hg.mozilla.org/integration/autoland/rev/fe1c6928ced3
Update OHTTP Relay Url used by Shopping component r=bholley,fchasen,Gijs
Comment 3•8 months ago
|
||
bugherder |
Status: NEW → RESOLVED
Closed: 8 months ago
status-firefox132:
--- → fixed
Resolution: --- → FIXED
Target Milestone: --- → 132 Branch
Assignee | ||
Comment 4•8 months ago
|
||
For consistency with other uses of OHTTP in Firefox, update the URL. Both old
and new URL are aliases for the same server so behaviours are the same.
Original Revision: https://phabricator.services.mozilla.com/D220618
Updated•8 months ago
|
Attachment #9424703 -
Flags: approval-mozilla-beta?
Comment 5•8 months ago
|
||
beta Uplift Approval Request
- User impact if declined: Different uses of OHTTP within Firefox will send traffic to different URLs when a shared name has slightly better privacy properties. Uplifting this to 131 means consistent behaviour if the PPA experiment launches in the 131 cycle.
- Code covered by automated testing: yes
- Fix verified in Nightly: yes
- Needs manual QE test: no
- Steps to reproduce for manual QE testing: N/A
- Risk associated with taking this patch: Low
- Explanation of risk level: The two URLs direct to the same server, and this change has been working fine in nightly for a week.
- String changes made/needed: None
- Is Android affected?: yes
Updated•8 months ago
|
Attachment #9424703 -
Flags: approval-mozilla-beta? → approval-mozilla-beta+
Updated•8 months ago
|
status-firefox131:
--- → fixed
You need to log in
before you can comment on or make changes to this bug.
Description
•