529986 - Add email address to individual reports when logged in

Status: RESOLVED FIXED

(Socorro :: General, task)

Description

[Samuel Sidler (old account; do not CC)]

Now that we have auth, we should add the email address a user submitted to individual reports when a person is logged in to crash-stats.

There's a bunch of cool things we could do with email addresses ("give me all email addresses for a given signature"), but this is likely the easiest thing to do and would help in the short term.

Note that I'm targeting this for 1.2 because I'd really like to get it and I imagine it's not hard to do, but if it is or if we're getting close, feel free to bump it.

Comment 1

[Austin King [:ozten]]

This existing in production in the comments tab of

https://crash-stats.mozilla.com/report/list?product=Firefox&version=Firefox%3A3.5.5&query_search=signature&query_type=exact&query=&date=&range_value=1&range_unit=weeks&do_query=1&signature=NPSWF32.dll%400x17ba9f

https://crash-stats.mozilla.com/report/index/6e26a95c-4ced-4d8c-ab10-caef22091118

caveat: report/index displays many comments/email addresses so you have to look at report details tab and then search down in the comments tab to a match.

Comment 2

[Austin King [:ozten]]

Ouch doing the opposite is painful - on report/list finding the link to that report from a comment with an email address.

Comment 3

[Austin King [:ozten]]

Attachment: First attempt, Adds the email address for authorized users

This patch adds email to the data pulled from the database.
If the user is logged in, then we will display the email address in the details pane.

Additionally, in the comments pane it linkfies the Submission date to the crash detail page.

Comment 4

[Ryan Snyder [:ryansnyder] [:rsnyder] [:rysny]]

Comment on attachment 414620 [details] [diff] [review]
First attempt, Adds the email address for authorized users

Looks good. Glad to see you're forcing HTTPS on report/index. r+

Comment 5

[Austin King [:ozten]]

Commited r1519