1688203 - improve summary for report view urls in Matrix and elsewhere

Status: RESOLVED FIXED

(Socorro :: Webapp, enhancement, P2)

Description

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

When someone pastes a url for a crash report in Matrix, Matrix shows a summary like this:

mozilla::dom::FragmentOrElement::cycleCollection::Trace - Signature report - Mozilla Crash Reports
Mozilla Crash Reports Search Sign in Product Navigation Quick Navigation Select
Product Firefox FennecAndroid Focus FirefoxReality Fenix GeckoView GeckoViewExample
ReferenceBrowser Thunderbird

That's not useful. It'd be better if it showed a summary of the crash report.

Comment 1

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

I think these should show up in the summary:

• crash signature
• crash id
• submitted_timestamp
• product
• channel
• version
• buildid

MDN has an article. I'll follow that and see how that goes:

author, description: https://developer.mozilla.org/en-US/docs/Learn/HTML/Introduction_to_HTML/The_head_metadata_in_HTML#adding_an_author_and_description

open graph data: https://developer.mozilla.org/en-US/docs/Learn/HTML/Introduction_to_HTML/The_head_metadata_in_HTML#other_types_of_metadata

Comment 2

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

Attachment: pr 5679: bug 1688203: improve crash report view summary in Matrix

Comment 3

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

willkg merged PR #5679: "bug 1688203: improve crash report view summary in Matrix" in 2b2810d.

Comment 4

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

Some thoughts: I skipped the submitted timestamp. The day is derivable from the crash id and I'm not sure how useful a datestamp is. Also, once we get this working for the crash report view summary, we should also improve the signature report view summary.

Comment 5

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

Jeff pointed out that Matrix uses this code to pull out the og information:

https://github.com/matrix-org/synapse/blob/ccb9616f26413db4b829536f454923f9edb75040/synapse/rest/media/v1/preview_url_resource.py#L715

Comment 6

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

Attachment: crash_report_preview_old.png

What the crash report view summary used to look like.

Comment 7

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

Attachment: crash_report_preview_new.png

What the crash report view looks like after these changes.

Comment 8

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

Attachment: pr 5680: bug 1688203: basic summary for signature report view

Comment 9

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

willkg merged PR #5680: "bug 1688203: basic summary for signature report view" in 2a0b14e.

Comment 10

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

Pushed this out just now in bug #1692041. Marking as FIXED.