"Learn more..." link on the error page for expired certificates links to TLS error reports content

VERIFIED FIXED in Firefox 45

Status

()

VERIFIED FIXED
3 years ago
3 years ago

People

(Reporter: mgoodwin, Assigned: mgoodwin)

Tracking

44 Branch
mozilla46
Points:
---
Dependency tree / graph

Firefox Tracking Flags

(firefox44- wontfix, firefox45+ verified, firefox46 verified)

Details

Attachments

(1 attachment)

(Assignee)

Description

3 years ago
Issue:
The "Learn More" link on the error page displayed for expired certs links to the TLS error report pages - which contains information only applicable to non-overridable errors.

Expected behavior:
The link should take the user to information pertinent to overridable TLS errors.

Steps to reproduce:
Visit https://expired.badssl.com/
https://support.mozilla.org/kb/what-does-your-connection-is-not-secure-mean is probably the most relevant and useful page at the moment (there is another page that covers similar content but goes into more depth (but uses out-of-date screenshots), so we might consider merging the two).

Comment 2

3 years ago
so it looks like we will have to merge the current content of https://support.mozilla.org/en-US/kb/connection-untrusted-error-message into https://support.mozilla.org/en-US/kb/what-does-your-connection-is-not-secure-mean and adapt it for 44 accordingly.

also need-infoing joni to make her aware of this issue...
Flags: needinfo?(jsavage)
(Assignee)

Comment 3

3 years ago
Can you let me know which URL you want to use so I can make the changes to aboutCertError.xhtml

It's worth noting that bug 1207130 (in progress) will mean reporting is available to both pages. With this in mind, maybe the content of https://support.mozilla.org/kb/tls-error-reports should be merged too?
(Assignee)

Comment 4

3 years ago
As discussed in IRC (with philipp), I'll also change the value in aboutNetError.xhtml
Version: unspecified → 44 Branch
(Assignee)

Comment 5

3 years ago
Created attachment 8709985 [details]
MozReview Request: Bug 1232258 - Fix 'Learn More' link in aboutCertError.xhtml and aboutNetError.xhtml r?Gijs

Review commit: https://reviewboard.mozilla.org/r/31623/diff/#index_header
See other reviews: https://reviewboard.mozilla.org/r/31623/
Attachment #8709985 - Flags: review?(gijskruitbosch+bugs)

Updated

3 years ago
Blocks: 1207107

Comment 6

3 years ago
Comment on attachment 8709985 [details]
MozReview Request: Bug 1232258 - Fix 'Learn More' link in aboutCertError.xhtml and aboutNetError.xhtml r?Gijs

302 past who named that page in bug 1207107 comment #0.
Attachment #8709985 - Flags: review?(gijskruitbosch+bugs) → review?(past)

Updated

3 years ago
Duplicate of this bug: 1229515

Comment 8

3 years ago
[Tracking Requested - why for this release]:
UI should point the user to the right place and not be confusing.
Blocks: 1218971
status-firefox44: --- → unaffected
status-firefox45: --- → affected
status-firefox46: --- → affected
tracking-firefox45: --- → ?

Comment 9

3 years ago
So hg.m.o would like to pretend that bug 1207107 only landed on 45, but it was uplifted. So we're about to ship this. I don't know what our odds are to still get this into 44.

It seems like the more sensible option for 44 would be to make the SUMO page just redirect to the thing we want. (And potentially include any other info that tls-error-report has but that page doesn't).

Joni/Mark, can we do that? Dumb idea? (should probably be a separate bug for tracking reasons if we do it)
status-firefox44: unaffected → affected
Flags: needinfo?(mgoodwin)
(Assignee)

Comment 10

3 years ago
(In reply to :Gijs Kruitbosch from comment #9)
> Joni/Mark, can we do that? Dumb idea? (should probably be a separate bug for
> tracking reasons if we do it)

In terms of the content, no. Everything that's on tls-error-report can (and given the new UI, probably should) be in the other page.
Flags: needinfo?(mgoodwin)
(Assignee)

Comment 11

3 years ago
(In reply to Mark Goodwin [:mgoodwin] from comment #10)
> In terms of the content, no.

To be clear, that's "no, it's not a dumb idea"
Comment on attachment 8709985 [details]
MozReview Request: Bug 1232258 - Fix 'Learn More' link in aboutCertError.xhtml and aboutNetError.xhtml r?Gijs

https://reviewboard.mozilla.org/r/31623/#review28343
Attachment #8709985 - Flags: review?(past) → review+

Comment 13

3 years ago
[Tracking Requested - why for this release]:
in addition to comment #8 adding tracking flag for 44 as well
tracking-firefox44: --- → ?

Comment 14

3 years ago
a first version of https://support.mozilla.org/en-US/kb/what-does-your-connection-is-not-secure-mean with some more relevant content is now public...
(Assignee)

Comment 16

3 years ago
https://hg.mozilla.org/integration/mozilla-inbound/rev/d5c2df6407e6db8503ed37fbdb002c5b0dadfa7e
Bug 1232258 - Fix 'Learn More' link in aboutCertError.xhtml and aboutNetError.xhtml r=past
(Assignee)

Updated

3 years ago
Assignee: nobody → mgoodwin

Comment 17

3 years ago
bugherder
https://hg.mozilla.org/mozilla-central/rev/d5c2df6407e6
Status: NEW → RESOLVED
Last Resolved: 3 years ago
status-firefox46: affected → fixed
Resolution: --- → FIXED
Target Milestone: --- → mozilla46

Comment 18

3 years ago
we have put a redirect in place on sumo now - maybe it would be good to get the fix uplifted to firefox 45 nevertheless.
Flags: needinfo?(jsavage)
Gijs, do we need to uplift this to beta45? It's a wontfix for fx44 with a hard-coded (temp) redirect on sumo for 44.
status-firefox44: affected → wontfix
tracking-firefox44: ? → -
Flags: needinfo?(gijskruitbosch+bugs)

Comment 20

3 years ago
Comment on attachment 8709985 [details]
MozReview Request: Bug 1232258 - Fix 'Learn More' link in aboutCertError.xhtml and aboutNetError.xhtml r?Gijs

(In reply to Ritu Kothari (:ritu) from comment #19)
> Gijs, do we need to uplift this to beta45? It's a wontfix for fx44 with a
> hard-coded (temp) redirect on sumo for 44.

Yes, also because esr. Normally we ask the patch author or the reviewer to do an uplift request, but I think Mark will forgive me if I do it for him... :-)

Approval Request Comment
[Feature/regressing bug #]: bug 1207107
[User impact if declined]: irrelevant/confusing information shown when clicking "learn more"
[Describe test coverage new/current, TreeHerder]: nope, just changing a link
[Risks and why]: none, just changing a link from one href to another, already baked for a bit
[String/UUID change made/needed]: no, the link was and is hardcoded, so no l10n changes. SUMO translations should be fine as the alternative page has existed for a while now, and anyway, that's not relevant for uplift.
Flags: needinfo?(gijskruitbosch+bugs)
Attachment #8709985 - Flags: approval-mozilla-beta?
Comment on attachment 8709985 [details]
MozReview Request: Bug 1232258 - Fix 'Learn More' link in aboutCertError.xhtml and aboutNetError.xhtml r?Gijs

Fix the link. Taking it. Should be in 45 beta2
Attachment #8709985 - Flags: approval-mozilla-beta? → approval-mozilla-beta+

Comment 22

3 years ago
bugherderuplift
https://hg.mozilla.org/releases/mozilla-beta/rev/5e37eea7c669
status-firefox45: affected → fixed
QA Whiteboard: [good first verify]
tracking-firefox45: ? → +
Verified as fixed in Firefox Stable 45 Debian 8 GNU/Linux 

Verified as Fixed in Firefox Beta 46.0b1 Debian 8 GNU/Linux
Status: RESOLVED → VERIFIED
QA Whiteboard: [good first verify] → [good first verify][bugday-20160316]
status-firefox45: fixed → verified
status-firefox46: fixed → verified
You need to log in before you can comment on or make changes to this bug.