Closed Bug 2055787 Opened 1 month ago Closed 1 month ago

Update documentation (esp SUMO) for new captive portal detection domain

Categories

(support.mozilla.org :: Knowledge Base Content, task)

Tracking

(Not tracked)

RESOLVED FIXED

People

(Reporter: simonf, Assigned: seburo3)

References

Details

(Keywords: sumo-contributors)

https://support.mozilla.org/en-US/kb/captive-portal and https://support.mozilla.org/en-US/kb/domains-allow-firefox include detectportal.firefox.com
We are moving this to firefox-portal-detection.com so we should add that everywhere we're talking about detectportal.firefox.com. We need to keep both for a while because old Firefox versions will keep talking to the old endpoint for a long time.

Component: Operations: Miscellaneous → Knowledge Base Content
Product: Cloud Services → support.mozilla.org
Summary: Update documentation (esp SUMO) for new domain → Update documentation (esp SUMO) for new captive portal detection domain
Assignee: nobody → kelimutu
Status: NEW → ASSIGNED

Assigning to Paul

Assignee: kelimutu → seburo3

Maybe it would be useful to add version tags, like we usually do on SUMO? What Firefox version will first use the new address?

I reached out to the requester directly about this - using both URL is not a problem in this instance.

Shouldn't we use "or" instead of "and" in the https://support.mozilla.org/en-US/kb/captive-portal article, when using both URLs?

See also https://support.mozilla.org/en-US/kb/captive-portal/revision/344903 Created: Aug 1, 2026, 2:48:28 PM (review pending)
Creator: Valentin Comment: actually, the captive portal loads are explicitly loaded over HTTP, not HTTPS

Thanks, Paul.

I've reviewed and approved the https://support.mozilla.org/en-US/kb/domains-allow-firefox/history article.

@Alice would you like to take the review for the other one since you don't have an outstanding revision there?

Flags: needinfo?(alice.wyman)

(In reply to Kelimutu [:kiki] from comment #8)

Thanks, Paul.

I've reviewed and approved the https://support.mozilla.org/en-US/kb/domains-allow-firefox/history article.

@Alice would you like to take the review for the other one since you don't have an outstanding revision there?

I approved the revision by Valentin to the https://support.mozilla.org/en-US/kb/captive-portal article.

Flags: needinfo?(alice.wyman)

@Alice Thanks, Alice. In the future, could you please make sure to review the oldest pending revision before approving a newer one?

It's possible that you've already reviewed the content eithout taking action on the actual revision. However, reviewing revisions in order helps ensure contributors don't feel that their work has been overlooked. It also keeps our pending revision queue accurate, so our backlog and metrics reflect work that genuinely still needs attention.

For reference, see the Reviewing multiple revisions section of the guidelines:
https://support.mozilla.org/en-US/kb/article-review-guidelines#w_reviewing-multiple-revisions

@all Thanks again for your collaboration here. I'm closing this ticket as resolved now.

Status: ASSIGNED → RESOLVED
Closed: 1 month ago
Resolution: --- → FIXED

(In reply to Kelimutu [:kiki] from comment #10)

@Alice Thanks, Alice. In the future, could you please make sure to review the oldest pending revision before approving a newer one?

It's possible that you've already reviewed the content eithout taking action on the actual revision. However, reviewing revisions in order helps ensure contributors don't feel that their work has been overlooked. It also keeps our pending revision queue accurate, so our backlog and metrics reflect work that genuinely still needs attention.

For reference, see the Reviewing multiple revisions section of the guidelines:
https://support.mozilla.org/en-US/kb/article-review-guidelines#w_reviewing-multiple-revisions

Kiki, Regarding https://support.mozilla.org/en-US/kb/captive-portal/history and the multiple revisions: I deferred my own pending revision, then approved the latest revision by Valentin a few days later but left the oldest pending revision by Paul "unreviewed", even though I did look at it.

You said to review the oldest pending revision before approving a newer one and referred me to the guidelines. By "review" I take that to mean, either defer or approve it. I agree, before approving the latest revision by Valentin I should have deferred Paul's revision, just like I deferred my own revision, since both had incorrect content. You are right, I should not have left it pending. The guidelines just say (quote) When there are multiple revisions pending for approval, make sure to check all of the revisions starting from the oldest one before making a decision. I think that it should be changed from "check" to "review (defer or approve)" to avoid confusion, since I did check Paul's revision but didn't take any review action.

(In reply to Denys [:denyshon] from comment #4)

Maybe it would be useful to add version tags, like we usually do on SUMO? What Firefox version will first use the new address?

According to bug 2049252 captive portal detection will move to the new domain (firefox-portal-detection.com) in Firefox version 155.

Will we still need to support the old domain for users of either of the two ESR versions?

I'd defer that question to Simon.

@Simon can you help answer both questions from Alice and Paul? The Captive portal detection aritcle is one of the most visited articles on SUMO (if not the highest), so providing the information so we can update the article accordingly is super important for us.

Flags: needinfo?(sfriedberger)

The old domains will still be in use by old Firefox versions and by ESRs. We should list them both in SUMO until we decide that the service running at detectportal.firefox.com has become unused and decommission it.

Flags: needinfo?(sfriedberger)

Got it! I just udpated the article to make it display detectportal.firefox.com for users below 155 and firefox-portal-detection.com for users on 155 and above.

You need to log in before you can comment on or make changes to this bug.