Closed
Bug 788049
Opened 12 years ago
Closed 12 years ago
[socorro-crashstats] implement /report/index
Categories
(Socorro :: Webapp, task)
Socorro
Webapp
Tracking
(Not tracked)
RESOLVED
FIXED
People
(Reporter: rhelmer, Assigned: rhelmer)
References
()
Details
(Whiteboard: https://github.com/mozilla/socorro-crashstats/issues/9)
Took a first stab at this:
https://github.com/rhelmer/socorro-crashstats/compare/issue-9-implement-report-index
I think we can get this ready except for:
correlations
waiting on https://bugzilla.mozilla.org/show_bug.cgi?id=650904 just like issue #8
authentication
we should leave the private data hidden until we have a general solution.. this should probably be it's own issue
Assignee | ||
Comment 1•12 years ago
|
||
We also need to pull in the fields specified in https://bugzilla.mozilla.org/show_bug.cgi?id=723297 and https://bugzilla.mozilla.org/show_bug.cgi?id=770078
Assignee | ||
Comment 2•12 years ago
|
||
Fennec needs to be rewritten to "Firefox for Android" in the "File a bug" link per https://bugzilla.mozilla.org/show_bug.cgi?id=770245
Assignee | ||
Comment 3•12 years ago
|
||
/report/index needs to be able to line up source code in the stack trace, e.g.:
https://crash-stats.mozilla.com/report/index/3ee38c20-01e4-4fcf-bc53-c2b4d2120826#frames
Assignee | ||
Comment 4•12 years ago
|
||
(In reply to Robert Helmer [:rhelmer] from comment #3)
> /report/index needs to be able to line up source code in the stack trace,
> e.g.:
> https://crash-stats.mozilla.com/report/index/3ee38c20-01e4-4fcf-bc53-
> c2b4d2120826#frames
This currently tries to link to files in the object dir ("obj-firefox"), should Socorro try to be smarter about this?
Status: NEW → ASSIGNED
Assignee | ||
Comment 5•12 years ago
|
||
(In reply to Robert Helmer [:rhelmer] from comment #1)
> We also need to pull in the fields specified in
> https://bugzilla.mozilla.org/show_bug.cgi?id=723297 and
> https://bugzilla.mozilla.org/show_bug.cgi?id=770078
Also bug 669108
Assignee | ||
Comment 6•12 years ago
|
||
(In reply to Robert Helmer [:rhelmer] from comment #5)
> (In reply to Robert Helmer [:rhelmer] from comment #1)
> > We also need to pull in the fields specified in
> > https://bugzilla.mozilla.org/show_bug.cgi?id=723297 and
> > https://bugzilla.mozilla.org/show_bug.cgi?id=770078
>
> Also bug 669108
Instead of playing whack-a-mole on these, I went over https://github.com/lauraxt/socorro/blob/a6159c603c332ef27b4c7f95e88624930bdecbc7/webapp-php/application/views/report/index.php and implemented all fields (and even showing/hiding based on whether they are set)
Assignee | ||
Comment 7•12 years ago
|
||
(In reply to Robert Helmer [:rhelmer] from comment #6)
> (In reply to Robert Helmer [:rhelmer] from comment #5)
> > (In reply to Robert Helmer [:rhelmer] from comment #1)
> > > We also need to pull in the fields specified in
> > > https://bugzilla.mozilla.org/show_bug.cgi?id=723297 and
> > > https://bugzilla.mozilla.org/show_bug.cgi?id=770078
> >
> > Also bug 669108
>
> Instead of playing whack-a-mole on these, I went over
> https://github.com/lauraxt/socorro/blob/
> a6159c603c332ef27b4c7f95e88624930bdecbc7/webapp-php/application/views/report/
> index.php and implemented all fields (and even showing/hiding based on
> whether they are set)
Oops imts https://github.com/mozilla/socorro/blob/master/webapp-php/application/views/report/index.php
Comment 8•12 years ago
|
||
(In reply to Robert Helmer [:rhelmer] from comment #4)
> This currently tries to link to files in the object dir ("obj-firefox"),
> should Socorro try to be smarter about this?
In theory, it may be a good idea to not link files like that, but I don't think a lot of dev time should be invested in that.
Assignee | ||
Comment 9•12 years ago
|
||
Comment 10•12 years ago
|
||
(In reply to Robert Helmer [:rhelmer] from comment #4)
> This currently tries to link to files in the object dir ("obj-firefox"),
> should Socorro try to be smarter about this?
No, that's a bug on the symbol dumping side, bug 462159. I think Socorro should just blindly accept what we feed it.
Assignee | ||
Updated•12 years ago
|
Status: ASSIGNED → RESOLVED
Closed: 12 years ago
Resolution: --- → FIXED
Assignee | ||
Comment 11•12 years ago
|
||
Updated the dev server, I already see a few little problems but overall seems to work:
http://crash-stats-new-dev.allizom.org/report/index/ec227a66-ad6e-4925-bb09-090f52120912
I will file followup bug(s).
You need to log in
before you can comment on or make changes to this bug.
Description
•