Thunderbird crash-stats related endpoint changes
Categories
(Thunderbird :: General, task, P2)
Tracking
(thunderbird_esr78 wontfix, thunderbird90 fixed)
People
(Reporter: mkmelin, Assigned: rjl)
References
Details
Attachments
(1 file)
(deleted),
text/x-phabricator-request
|
wsmwk
:
approval-comm-beta+
|
Details |
See bug 1608971.
We should change the endpoints we point to to use the Thunderbird corresponding ones. There is
pref("breakpad.reportURL", "https://crash-stats.mozilla.com/report/index/");
pref("toolkit.crashreporter.infoURL",
"https://www.mozilla.org/thunderbird/legal/privacy/#crash-reporter");
And also ServerURL at https://searchfox.org/mozilla-central/rev/a707541ff423ade0d81cef6488e6ecfa09273886/build/application.ini.in#50 (which can be overriden for testing with MOZ_CRASHREPORTER_URL)
I think application.ini.in should be changed to take a variable for the serverURL hostname.
Reporter | ||
Updated•5 years ago
|
Comment 1•5 years ago
|
||
How's this going to work from the Mozilla side?
I have CNAMED https://crash-stats.thunderbird.net and https://crash-reports.thunderbird.net over to the respective Mozilla domains, but of course that will not work unless they adjust their SSL certs for those domains.
Reporter | ||
Comment 2•4 years ago
|
||
The current crash-stats certificate expires 2021-02-17. I guess when renewing that would be a good time to add the thunderbird names.
Comment 4•4 years ago
|
||
Magnus: I'm not sure I understand what that means. Can you elaborate?
Reporter | ||
Comment 5•4 years ago
|
||
Sure, the SSL certificates for crash-stats.mozilla.org (and crash-reports) are only valid for that hostname. So until the certificate is also valid for a Thunderbird related endpoint (comment 1) we could not switch clients over to the new endpoints.
Once we switch to something else we'd point the Thunderbird endpoints to what that is, but without adjusting the certificates that would have to happen "over night" for all clients which seems impossible. If the certificates are adjusted we can switch this client side already, then we just point the crash-stats.thunderbird.net domain elsewhere once we're ready.
Comment 6•4 years ago
|
||
What domain name do you want to be added to the crash-stats.mozilla.org (the webapp) certificate?
What domain name do you want added to the crash-reports.mozilla.com (the crash ingestion pipeline collector) certificate?
Reporter | ||
Comment 7•4 years ago
|
||
(In reply to Will Kahn-Greene [:willkg] ET needinfo? me from comment #6)
What domain name do you want to be added to the crash-stats.mozilla.org (the webapp) certificate?
crash-stats.thunderbird.net
What domain name do you want added to the crash-reports.mozilla.com (the crash ingestion pipeline collector) certificate?
crash-reports.thunderbird.net
Comment 8•4 years ago
|
||
I talked to my ops person. It's really complicated and a lot of time and effort to add the domains you want added to our SSL certificates. Further, we reissued our certificates already.
Is it possible to do something like this?:
-
For crash-stats (the webapp), I think you should set up a basic nginx server that takes incoming crash-stats.thunderbird.net urls and redirects them to crash-stats.mozilla.org.
-
For crash-reports (the crash ingestion pipeline collector), I suggest you set up a tee server that takes an incoming HTTP POST, does the HTTP POST on behalf of the client to crash-reports.mozilla.com, gets back the response, and then sends the response to the client.
Comment 9•4 years ago
|
||
(In reply to Will Kahn-Greene [:willkg] ET needinfo? me from comment #8)
Is it possible to do something like this?:
- For crash-stats (the webapp), I think you should set up a basic nginx server that takes incoming crash-stats.thunderbird.net urls and redirects them to crash-stats.mozilla.org.
Redirects are trivial so sure.
- For crash-reports (the crash ingestion pipeline collector), I suggest you set up a tee server that takes an incoming HTTP POST, does the HTTP POST on behalf of the client to crash-reports.mozilla.com, gets back the response, and then sends the response to the client.
Sorry but this suggestion seems unreasonable to me. You're saying that setting up a bandwidth/request intensive forwarding service that can't use caching is easier than changing an SSL cert? Something is seriously wrong if that's the case.
I know that Firefox/Thunderbird should be able to follow 307 redirects for POSTs, but I don't know if this applies to the crash reporter service...
Reporter | ||
Comment 10•4 years ago
|
||
For #1 it's easy.
For #2 we'll investigate if we can just HTTP 307 redirect it appropriately.
Comment 11•3 years ago
|
||
New URL should be: https://crash-reports.thunderbird.net
Assignee | ||
Comment 12•3 years ago
|
||
Sets an alternate crashreporter url used by the breakpad client and the
Javascript submitter.
Updated•3 years ago
|
Assignee | ||
Updated•3 years ago
|
Updated•3 years ago
|
Comment 13•3 years ago
|
||
Comment on attachment 9226903 [details]
Bug 1628329 - Set MOZ_CRASHREPORTER_URL at build time. r=sancus
[Approval Request Comment]
Risk to taking this patch (and alternatives if risky):
This changes the URL target for the crash reporter only, which is low risk as it doesn't affect any other client code.
Comment 14•3 years ago
|
||
Pushed by thunderbird@calypsoblue.org:
https://hg.mozilla.org/comm-central/rev/d0f8efdda8e5
Set MOZ_CRASHREPORTER_URL at build time. r=sancus
Comment 15•3 years ago
|
||
Comment on attachment 9226903 [details]
Bug 1628329 - Set MOZ_CRASHREPORTER_URL at build time. r=sancus
[Triage Comment]
Approved for beta
Assignee | ||
Updated•3 years ago
|
Comment 16•3 years ago
|
||
bugherder uplift |
Thunderbird 90.0b3
https://hg.mozilla.org/releases/comm-beta/rev/2102fadbb685
Comment 17•3 years ago
|
||
Comment on attachment 9226903 [details]
Bug 1628329 - Set MOZ_CRASHREPORTER_URL at build time. r=sancus
[Approval Request Comment]
We should get this into the next ESR78 so that crash reports don't bounce when Socorro stops accepting Thunderbird reports.
Comment 18•3 years ago
|
||
Comment on attachment 9226903 [details]
Bug 1628329 - Set MOZ_CRASHREPORTER_URL at build time. r=sancus
[Triage Comment]
Approved for esr78
Assignee | ||
Comment 19•3 years ago
|
||
Comment on attachment 9226903 [details]
Bug 1628329 - Set MOZ_CRASHREPORTER_URL at build time. r=sancus
Not possible with esr78
Assignee | ||
Updated•3 years ago
|
Description
•