Closed
Bug 1433500
Opened 5 years ago
Closed 1 year ago
detectportal.firefox.com should return something informative for "real" user agents.
Categories
(Core :: Networking, enhancement, P3)
Core
Networking
Tracking
()
RESOLVED
FIXED
89 Branch
Tracking | Status | |
---|---|---|
firefox89 | --- | fixed |
People
(Reporter: mhoye, Assigned: valentin)
References
Details
(Whiteboard: [necko-triaged])
Attachments
(1 file)
Right now, browsing to detectportal.firefox.com returns only "success". For some users - particularly non-english-native-speaking users - the purpose of detectportal.firefox.com may be unclear, and I've heard a report that people are concerned about the nature of that ping from a privacy and security perspective. I propose that we append a (shortened?) link to an appropriate SUMO page about what detectportal.firefox.com is and does to success.txt.
![]() |
||
Comment 1•5 years ago
|
||
Tentatively giving to Valentin as a maintainer of that code.
Assignee: nobody → valentin.gosu
Priority: -- → P3
Whiteboard: [necko-triaged]
Assignee | ||
Comment 3•2 years ago
|
||
Steps:
- Add a new file
canonical.html
to the storage bucket containing<meta http-equiv="refresh" content="0;url=http://mzl.la/cp"/>
- Change the
captivedetect.canonicalContent
to<meta http-equiv="refresh" content="0;url=http://mzl.la/cp"/>
- Change
captivedetect.canonicalURL
tohttp://detectportal.firefox.com/canonical.html
- Make
http://mzl.la/cp
redirect to a Mozilla support page explaining captive portals.
Follow-up steps:
- Change domain from
detectportal.firefox.com
to a TLD so we don't have to worry about HSTS on firefox.com - Change
network.connectivity-service.IPv4.url
and http://detectportal.firefox.com/canonical.html?ipv4` - Change
network.connectivity-service.IPv6.url
and http://detectportal.firefox.com/canonical.html?ipv6` - Eventually remove success.txt when it is not used (or simply decomission detectportal.firefox.com)
Assignee | ||
Comment 4•2 years ago
|
||
This change constitutes a way we can redirect users to an actual page that
explains what captive portals are and why we are making these requests.
Normally users should not see this page, as we only compare the contents
of a small html file. The meta redirect only happens when loaded in a
page.
The http://mzl.la/cp redirect is controlled via the bit.ly URL shortner.
Updated•1 year ago
|
Attachment #9193231 -
Attachment description: Bug 1433500 - detectportal.firefox.com should return something informative for users r=nhnt11,Gijs → Bug 1433500 - detectportal.firefox.com should return something informative for users r=nhnt11
Pushed by valentin.gosu@gmail.com: https://hg.mozilla.org/integration/autoland/rev/153bb20062aa detectportal.firefox.com should return something informative for users r=nhnt11
Comment 6•1 year ago
|
||
Backed out for causing failure on test_captive_portal_service.js
Backout link: https://hg.mozilla.org/integration/autoland/rev/b532729e3b07852d862528e03b597974948d7b7e
Failure log: https://treeherder.mozilla.org/logviewer?job_id=334538770&repo=autoland&lineNumber=2970
Flags: needinfo?(valentin.gosu)
Assignee | ||
Updated•1 year ago
|
Flags: needinfo?(valentin.gosu)
Pushed by valentin.gosu@gmail.com: https://hg.mozilla.org/integration/autoland/rev/2fcbc72daec5 detectportal.firefox.com should return something informative for users r=nhnt11,extension-reviewers,zombie
Comment 8•1 year ago
|
||
bugherder |
Status: NEW → RESOLVED
Closed: 1 year ago
status-firefox89:
--- → fixed
Resolution: --- → FIXED
Target Milestone: --- → 89 Branch
You need to log in
before you can comment on or make changes to this bug.
Description
•