966866 - Connected to a site which is run by 'null/null' certificate

Status: VERIFIED FIXED

(Firefox for Android Graveyard :: General, defect)

Description

[Aaron Train [:aaronmt]]

Attachment: screenshot.png

See screenshot

The site has an AddTrust External CA Root and a valid Comodo SSL CA 

https://www.davidstea.com/

In Nightly, this is shown, in Aurora/Beta/Release this is not.

What is going on here?

Comment 1

[Kevin Brosnan [Ex-Mozilla]]

I get the insecure content warning on all releases using

1) visit https://www.davidstea.com in a private window
3) Select EN-ca or FR-ca

Selecting EN-us does not result in the mixed content warning. Invalid?

Comment 3

[Aaron Train [:aaronmt]]

Not sure how this can be invalid. Look at the screenshot.

Comment 4

[Aaron Train [:aaronmt]]

URL from Teodora's duplicate bug: https://ie.microsoft.com/testdrive/browser/mixedcontent/assets/woodgrove.htm

Comment 5

[Teodora Vermesan (:TeoVermesan)]

Regression window-wanted:
mc:
not affected build: 12-12-2013
affected build: 13-12-2013  
-pushlog:
http://hg.mozilla.org/mozilla-central/pushloghtml?fromchange=1ad9af3a2ab8&tochange=8b5875dc7e31

Comment 6

[Aaron Train [:aaronmt]]

I wonder if this is more accidental fallout from bug 945212 which is in that massive range.

Comment 7

[:Margaret Leibovic]

(In reply to Aaron Train [:aaronmt] from comment #6)
> I wonder if this is more accidental fallout from bug 945212 which is in that
> massive range.

Sounds likely.

Comment 8

[Lucas Rocha (:lucasr)]

Attachment: Correctly update SiteIdentityPopup for MIXED_CONTENT_BLOCKED (r=margaret)

Comment 9

[Lucas Rocha (:lucasr)]

Comment on attachment 8372359 [details] [diff] [review]
Correctly update SiteIdentityPopup for MIXED_CONTENT_BLOCKED (r=margaret)

MIXED_CONTENT_BLOCKED should also hide the identity UI from the site identity popup.

Comment 10

[Lucas Rocha (:lucasr)]

https://hg.mozilla.org/integration/fx-team/rev/5a6305a56957

Comment 11

[Lucas Rocha (:lucasr)]

Comment on attachment 8372359 [details] [diff] [review]
Correctly update SiteIdentityPopup for MIXED_CONTENT_BLOCKED (r=margaret)

[Approval Request Comment]
Bug caused by (feature/regressing bug #): bug 945212 and bug 944533
User impact if declined: Broken mixed content blocking dialog
Testing completed (on m-c, etc.): Just landed in m-c, let's wait a couple days and then land it in Fx29.
Risk to taking this patch (and alternatives if risky): None, tiny patch to cover all mixed content blocking states. 
String or IDL/UUID changes made by this patch: n/a

Comment 12

[Wes Kocher (:KWierso) (Not reading bugmail; email directly if needed)]

https://hg.mozilla.org/mozilla-central/rev/5a6305a56957

Comment 13

[Lucas Rocha (:lucasr)]

https://hg.mozilla.org/releases/mozilla-aurora/rev/ae766edb572b

Comment 14

[Teodora Vermesan (:TeoVermesan)]

Verified fixed on:
Build: Firefox for Android 30.0a1 (2014-02-09) 
Device: LG Nexus 4
OS: Android 4.4

Comment 15

[Teodora Vermesan (:TeoVermesan)]

Verified fixed on:
Build: Firefox for Android 29.0a2 (2014-03-05) 
Device: Alcatel One Touch 
OS: Android 4.1.2