Allow specifying a different ServerURL in application.ini.in
Categories
(Firefox Build System :: General, enhancement)
Tracking
(firefox88 fixed)
Tracking | Status | |
---|---|---|
firefox88 | --- | fixed |
People
(Reporter: ewong, Assigned: ewong)
References
(Blocks 1 open bug)
Details
Attachments
(2 files, 3 obsolete files)
(deleted),
patch
|
Details | Diff | Splinter Review | |
(deleted),
text/x-phabricator-request
|
Details |
Socorro is phasing out support for ingesting crash reports from SeaMonkey (Bug 1608976) and Thunderbird (bug 1604848).
Both products use application.ini.in's ServerURL's entry to specify where to send the crash stats as stated:
Since crash-reports.mozilla.com is hard-coded, it would make migrating off this server hard.
I can see a few options though it's been a long time since i've touched the build config so I don't know the far-reaching affects of this change.
in confvars.sh:
MOZ_CRASHREPORTER_SERVER=https://<server>/<path>
in application.ini.in:
ServerURL=@MOZ_CRASHREPORTER_SERVER?id....
Or even have the whole URL defined in confvars.sh, i.e.
MOZ_CRASHREPORTER_URL=<full url including id, buildid, version>
in application.ini.in:
ServerURL=@MOZ_CRASHREPORTER_URL@
Assignee | ||
Comment 1•4 years ago
|
||
a wip.. (had the intention to set the whole URL, but build_id isn't available during that point (* I think... could be wrong..*)
so in TB and/or SM, in confvafs.sh, one could specify MOZ_CRASHREPORTER_URL to point to the
correct url.
Comment 2•4 years ago
|
||
Comment 3•4 years ago
|
||
See bug 1628329.
Assignee | ||
Comment 4•4 years ago
|
||
(In reply to Mike Hommey [:glandium] from comment #2)
Comment on attachment 9186187 [details] [diff] [review]
wip:Review of attachment 9186187 [details] [diff] [review]:
Please use phabricator
https://firefox-source-docs.mozilla.org/contributing/contribution_quickref.
html#to-submit-a-patch
The UI in bugzilla doesn't allow to write proper reviews anymore. I'm
surprised you were able to set a review flag at all.
A bug?
Please don't set the default crash reporter url in confvars.sh, because your
change, for instance, excludes mobile/android. It would be better to have
the default in configure. Also, please don't set this in old-configure. Use
python configure. The ideal place for this would be toolkit/moz.configure.
Thanks for the info. I haven't submitted a patch to core in a long time,
so a learning experience.
Just to clarify, toolkit/moz.configure is the only place? I don't need to
fiddle with build/moz.build?
Comment 5•4 years ago
|
||
(In reply to Edmund Wong (:ewong) from comment #4)
(In reply to Mike Hommey [:glandium] from comment #2)
Comment on attachment 9186187 [details] [diff] [review]
wip:Review of attachment 9186187 [details] [diff] [review]:
Please use phabricator
https://firefox-source-docs.mozilla.org/contributing/contribution_quickref.
html#to-submit-a-patch
The UI in bugzilla doesn't allow to write proper reviews anymore. I'm
surprised you were able to set a review flag at all.A bug?
A feature.
Please don't set the default crash reporter url in confvars.sh, because your
change, for instance, excludes mobile/android. It would be better to have
the default in configure. Also, please don't set this in old-configure. Use
python configure. The ideal place for this would be toolkit/moz.configure.Thanks for the info. I haven't submitted a patch to core in a long time,
so a learning experience.Just to clarify, toolkit/moz.configure is the only place? I don't need to
fiddle with build/moz.build?
You'll still have to, to pass the right things for application.ini
Comment 6•4 years ago
|
||
Thanks for the info. I haven't submitted a patch to core in a long time,
so a learning experience.
This doc is up to date:
https://firefox-source-docs.mozilla.org/contributing/contribution_quickref.html
Assignee | ||
Comment 7•4 years ago
|
||
Assignee | ||
Comment 8•4 years ago
|
||
Updated•4 years ago
|
Updated•4 years ago
|
Updated•4 years ago
|
Comment 9•4 years ago
|
||
There's a r+ patch which didn't land and no activity in this bug for 2 weeks.
:ewong, could you have a look please?
For more information, please visit auto_nag documentation.
Assignee | ||
Comment 10•4 years ago
|
||
@glandium, It's been a long time since I've pushed anything (can't push right now as my hg creds are still disabled). What needs to be done now?
Updated•4 years ago
|
Comment 11•4 years ago
|
||
Comment 12•4 years ago
|
||
Comment 13•4 years ago
|
||
bugherder |
https://hg.mozilla.org/mozilla-central/rev/bee2ef0fc0e7
https://hg.mozilla.org/mozilla-central/rev/2bfbb6e4c5dd
Comment 14•4 years ago
|
||
Backed out for sending crash reports to wrong host.
https://hg.mozilla.org/mozilla-central/rev/1c1dbf630893604844a607bfe87b73253267b01a
Comment 15•4 years ago
|
||
crash-stats.mozilla.com got used instead of crash-reports.mozilla.com and the host variable contains the trailing /
while it is also part of the submission url variable.
Assignee | ||
Comment 16•4 years ago
|
||
(In reply to Razvan Maries from comment #14)
Backed out for sending crash reports to wrong host.
https://hg.mozilla.org/mozilla-central/rev/1c1dbf630893604844a607bfe87b73253267b01a
My apologies. Will get this fixed.
Assignee | ||
Comment 17•4 years ago
|
||
Depends on D96322
Updated•4 years ago
|
Updated•4 years ago
|
Assignee | ||
Comment 18•4 years ago
|
||
Depends on D96498
Updated•4 years ago
|
Updated•4 years ago
|
Comment 19•4 years ago
|
||
ewong, had the chance to update the patch?
Assignee | ||
Comment 20•4 years ago
|
||
(In reply to Magnus Melin [:mkmelin] from comment #19)
ewong, had the chance to update the patch?
Yes, I have and I thought I had updated it. I'll take a look.
Assignee | ||
Comment 21•4 years ago
|
||
@glandium, I'm sorry for the delay in getting this bug done, but can you point out what other changes need to be done? I'm still not familiar with phabricator and I thought I had updated the patch. What else am I missing? Do I need a check-in needed? Thanks!
Comment 22•4 years ago
|
||
(In reply to Edmund Wong (:ewong) from comment #21)
Do I need a check-in needed?
Looks like it. Except if you have commit access level 3, in case you can land yourself.
Assignee | ||
Comment 23•4 years ago
|
||
(In reply to Mike Hommey [:glandium] from comment #22)
(In reply to Edmund Wong (:ewong) from comment #21)
Do I need a check-in needed?
Looks like it. Except if you have commit access level 3, in case you can land yourself.
Thanks for the help! It seems as if they've removed the checkin-needed (and I'm not sure if check-in needed-tb is what I should use).
Comment 24•4 years ago
|
||
Comment 25•4 years ago
|
||
bugherder |
Description
•