618185 - Update breakpad.reportURL URL to crash-stats.mozilla.org

Status: RESOLVED FIXED

(Toolkit :: Crash Reporting, task)

Description

[Reed Loden [:reed]]

The URL for crash-stats is changing in bug 618153 from crash-stats.mozilla.com to crash-stats.mozilla.org. Need to update all the various toolkit apps to use the new hostname.

Comment 1

[(not currently active) Ted Mielczarek]

I don't think we should bother changing the serverURL unless there's some real pressing concern. (It's set to crash-reports.mozilla.com, FWIW.) We can change breakpad.reportURL, though which is how we generate the links from about:crashes:
http://mxr.mozilla.org/mozilla-central/source/browser/app/profile/firefox.js#869

Comment 2

[Reed Loden [:reed]]

Ah, that's even better.

Comment 3

[Jesse Ruderman]

Also change it to https please :)

Comment 4

[(not currently active) Ted Mielczarek]

Why? This is strictly for generating the links within about:crashes.

Comment 5

[Jesse Ruderman]

For performance (the server redirects to https) and privacy (of the connection between the crash report and the user), mostly.

Comment 6

[gillh11]

I am interested in working on this bug, Can I get this assigned to me?

Comment 7

[(not currently active) Ted Mielczarek]

Sure! The relevant location in the source is:
http://hg.mozilla.org/mozilla-central/annotate/879038dcacb7/browser/app/profile/firefox.js#l939

Comment 8

[Abhishek Potnis [:avp]]

Hi,
Are you working on this bug?

Comment 9

[gillh11]

Sorry I don't think I can finish this bug

Comment 10

[Abhishek Potnis [:avp]]

Attachment: bug-618185-fix.diff

Comment 11

[(not currently active) Ted Mielczarek]

Comment on attachment 8394694 [details] [diff] [review]
bug-618185-fix.diff

Review of attachment 8394694 [details] [diff] [review]:
-----------------------------------------------------------------

This looks fine, thanks!

Comment 12

[amanjain110893]

is this bug fixed ?
actually I am newbie and I need to start from some bug
if its not fixed can someone assigned me this bug ?

Comment 13

[(not currently active) Ted Mielczarek]

This bug is ASSIGNED, which means someone is working on it. In fact it already has a reviewed patch, which means it's basically done.

Comment 14

[Reed Loden [:reed]]

(In reply to Ted Mielczarek [:ted.mielczarek] from comment #13)
> This bug is ASSIGNED, which means someone is working on it. In fact it
> already has a reviewed patch, which means it's basically done.

This can't land until the SSL cert on crash-stats.m.o is modified to include .m.o besides .m.c.

Comment 15

[(not currently active) Ted Mielczarek]

Thanks, didn't notice the dep there.

Comment 16

[David Nelson]

Hi,

Can I start work on this bug please?

Thanks
David

Comment 17

[Josh Matthews [:jdm]]

It looks like this bug is basically finished; we're just waiting on some website infrastructure changes before we can commit the patch up above.

Comment 18

[:Atoll]

(In reply to Josh Matthews [:jdm] from comment #17)
> It looks like this bug is basically finished; we're just waiting on some
> website infrastructure changes before we can commit the patch up above.

The infrastructure for .org is now in place and verified working.

Comment 19

[Gabriele Svelto [:gsvelto] (PTO)]

Ted, do we still want to make this change? The code still uses the .com version so either we switch to .org or close this.

Comment 20

[Will Kahn-Greene [:willkg] ET needinfo? me]

I would like it if we made this change. We've got bug #1322412 covering dropping the .com versions of things in the Socorro product. It'd be nice to start that ball rolling.

Comment 21

[Gabriele Svelto [:gsvelto] (PTO)]

OK, this is a trivial fix. The HTTPS certificate is the same for both .com and .org right?

Comment 22

[Will Kahn-Greene [:willkg] ET needinfo? me]

Miles: Can you answer the question about the certificate? ^^^

Comment 23

[Gabriele Svelto [:gsvelto] (PTO)]

It's currently pinned [1] to the certificate used by other services [2]. I assume it's the same but better safe than sorry.

[1] https://searchfox.org/mozilla-central/source/security/manager/tools/PreloadedHPKPins.json#213-214
[2] https://searchfox.org/mozilla-central/source/security/manager/tools/PreloadedHPKPins.json#73

Comment 24

[Miles Crabill [:miles]]

The certificate in place covers both crash-stats.mozilla.com and crash-stats.mozilla.org and those cert pins are correct.

Comment 25

[Gabriele Svelto [:gsvelto] (PTO)]

Attachment: Bug 618185 - Switch the default URL for sending crash reports to the *.mozilla.org domain

Comment 26

[Pulsebot]

Pushed by [e-mail]:
https://hg.mozilla.org/integration/autoland/rev/ec4b2e77d693
Switch the default URL for sending crash reports to the *.mozilla.org domain r=ted

Comment 27

[Will Kahn-Greene [:willkg] ET needinfo? me]

Hooray! Now we just need to wait a few years and we can drop the crash-stats.mozilla.com.

Thank you!

Comment 28

[Razvan Maries]

https://hg.mozilla.org/mozilla-central/rev/ec4b2e77d693