Closed Bug 1185088 Opened 4 years ago Closed 4 years ago

Enable SSL Extended Validation for Graphene

Categories

(Firefox OS Graveyard :: Runtime, defect)

ARM
Gonk (Firefox OS)
defect
Not set

Tracking

(Not tracked)

RESOLVED FIXED

People

(Reporter: bbondy, Assigned: bbondy)

References

Details

(Whiteboard: graphene-larch)

Attachments

(1 file)

I noticed that SSL EV is always set to false even for sites that support it in Graphene. 

It turns out that it's because of a hardcoded MOZ_NO_EV_CERTS=1 in confvars.sh.

I checked with evilpie who was in the hg annotate for this flag.  He mentioned that B2G lacked the UI for SSL EV and it may save space.  This seems like less of a concern for Graphene and I think it's better to have mozbrowsersecuritychange's detail return a correct result for the event's detail.extendedValidation.  I'll likely do another patch in a different bug later to add extra information as well which the UI can take advantage of. 

Patch incoming.
Attached patch sslev.diffSplinter Review
Attachment #8635507 - Flags: review?(paul)
Whiteboard: graphene-larch
Attachment #8635507 - Flags: review?(paul) → review+
url:        https://hg.mozilla.org/projects/larch/rev/3f97cfdaa08527dea2e9ece84303aea700d3b58d
changeset:  3f97cfdaa08527dea2e9ece84303aea700d3b58d
user:       Brian R. Bondy <netzen@gmail.com>
date:       Fri Jul 17 21:30:10 2015 -0400
description:
Bug 1185088 - Enable SSL Extended Validation for Graphene. r=paul

B2G lacked the UI for SSL EV and it may save space.  This seems like less of a
concern for Graphene and I think it's better to have mozbrowsersecuritychange's
detail return a correct result for the event's detail.extendedValidation.
Landed in mozilla-central via bug 1204965.
Status: NEW → RESOLVED
Closed: 4 years ago
Resolution: --- → FIXED
You need to log in before you can comment on or make changes to this bug.