Closed Bug 663055 Opened 13 years ago Closed 13 years ago

Mozilla nightly builds are not code signed

Categories

(Release Engineering :: General, defect, P3)

defect

Tracking

(Not tracked)

RESOLVED DUPLICATE of bug 509158

People

(Reporter: apenta, Unassigned)

References

()

Details

(Whiteboard: [signing])

User-Agent:       Mozilla/5.0 (compatible; MSIE 9.0; Windows NT 6.1; WOW64; Trident/5.0)
Build Identifier: 7.0a1

Mozilla products that offer nightly builds (firefox, thunderbird, etc) are not code signed. Code signing offers protection from tampering and make the nature of these nightly builds clear when the show up hosted on non-mozilla web properties.

Reproducible: Always

Steps to Reproduce:
1. Download nightly build
2. it's not code signed


Actual Results:  
downloaded file is not code signed

Expected Results:  
Mozilla ought to code sign all distributed installers/apps
Component: General → Release Engineering
Product: Mozilla Services → mozilla.org
QA Contact: general → release
Version: unspecified → other
Yep. This is conscious decision. If you think this is really important, please start a discussion in the mozilla.dev.planning newsgroup, and re-open this bug if it results in consensus.
Status: UNCONFIRMED → RESOLVED
Closed: 13 years ago
Resolution: --- → WONTFIX
Depends on: 509158
This is still being debated. You can follow along or join that conversation here: http://groups.google.com/group/mozilla.dev.planning/browse_thread/thread/16a26afbedf97637/#
Status: RESOLVED → REOPENED
Ever confirmed: true
Resolution: WONTFIX → ---
Status: REOPENED → NEW
Priority: -- → P3
Whiteboard: [signing]
One possible use case for signing Nightly builds is for bug 481815 (Provide a Windows service to update applications).

The service currently runs udpater.exe to perform an actual update.  
It would be nice to check that updater.exe is signed by us before executing it.
 
This extra check is not currently being implemented because Nightly builds are not signed by us.
I don't think this is a security blocker as you need to be an elevated process to replace updater.exe in our Program Files directory.

Since silent updates will not have a UAC prompt soon, it would be nice to have this extra check on behalf of a user who normally clicks to elevate.
Blocks: 481815
It seems a core security part of bug 481815 (P1) will depend on using signed bins.
To ensure testing on Nightly and Aurora first this makes this bug very important.

Otherwise bug 481815 would check the signed bin and fail on that check and would not use the service for updates.  So the task would only be tested once it reached a build with a signed bin by us.  Since bug 481815  is directly related to the update process, this is something that should get testing on Aurora and Nightly. 

I was wondering if you guys knew who could work on this or how we could move it along faster?
It is fine by me if we used a different certificate for Aurora and Nightly but I'd like it if Beta used the same certificate as the Release so there are no surprises on release day (I think Beta/Release signing is already the case).

All of this only matters to me in Windows using authenticode certificates as bug 481815 is only for Windows.
We're hoping to have the signing-on-demand done by the end of Q4. That should allow us to have a separate nightly signing server where we can sign windows nightlies with a different cert.
Chris, given the focus on Firefox updates and the need to get the change for bug 481815 landed is there anything your team can do to move up this work to earlier in 4Q?
(In reply to Lawrence Mandel [:lmandel] from comment #6)
> Chris, given the focus on Firefox updates and the need to get the change for
> bug 481815 landed is there anything your team can do to move up this work to
> earlier in 4Q?

It's my priority right now, but realistically I don't see this happening before late November.
Depends on: 696775
Depends on: 701069
We ended up doing this work in bug 509158.
Status: NEW → RESOLVED
Closed: 13 years ago13 years ago
Resolution: --- → DUPLICATE
Duping to the correct bug 509158 per comment 8.
Product: mozilla.org → Release Engineering
You need to log in before you can comment on or make changes to this bug.