1671654 - Add JavaException as a crash annotation

Status: RESOLVED FIXED

(Toolkit :: Crash Reporting, enhancement)

Description

[Roger Yang [:royang]]

Adding JavaException annotation for reporting stacktrace in a structured format.

Comment 1

[Roger Yang [:royang]]

Attachment: Data Review.txt

Comment 2

[Roger Yang [:royang]]

Try: https://treeherder.mozilla.org/#/jobs?repo=try&revision=78551fcbc5ad45ecd8d879737424fe05135d59c8

Comment 3

[Roger Yang [:royang]]

Attachment: Bug 1671654 - Add JavaException annotation for determining the cause of a Java exception. r=gsvelto!

Comment 4

[__oldboek]

Comment on attachment 9182064 [details]
Data Review.txt

Data Review Form (to be filled by Data Stewards)

1. Is there or will there be documentation that describes the schema for the ultimate data set in a public, complete, and accurate way?
   Yes, in CrashAnnotations.yaml.

2. Is there a control mechanism that allows the user to turn the data collection on and off?
   Yes,

3. If the request is for permanent data collection, is there someone who will monitor the data over time?
   Roger and the a-c team.

4. Using the category system of data types on the Mozilla wiki, what collection type of data do the requested measurements fall under?
   Type 1

5. Is the data collection request for default-on or default-off?
   Determined by the client

6. Does the instrumentation include the addition of any new identifiers (whether anonymous or otherwise; e.g., username, random IDs, etc. See the appendix for more details)?
   No

7. Is the data collection covered by the existing Firefox privacy notice?
   Yes

8. Does there need to be a check-in in the future to determine whether to renew the data?
   No

9. Does the data collection use a third-party collection tool?
   No

Comment 5

[Roger Yang [:royang]]

Attachment: Data Review.txt

Comment 6

[BugBot [:suhaib / :marco/ :calixte]]

There's a r+ patch which didn't land and no activity in this bug for 2 weeks.
:royang, could you have a look please?
For more information, please visit auto_nag documentation.

Comment 7

[Roger Yang [:royang]]

Sorry, I had to change the data review category about couple weeks ago. I'll ping the data reviewer again. Thanks

Comment 8

[__oldboek]

Hey Emily,

This request is Cat 3 so I wanted to escalate it and have you take a look. Thank you!

Comment 9

[Emily Litka]

Hi - sorry for the delay. What cat 3 data might be collected?

Thanks!

Comment 10

[Roger Yang [:royang]]

Hi Emily, The Cat 3 data is that the Exception Message might include URL information with possible PII information. Thanks,

Comment 11

[Emily Litka]

OK, thanks for explaining further. If this is new collection, to the extent this is applicable, can you please ensure it's disclosed here: https://firefox-source-docs.mozilla.org/toolkit/crashreporter/crashreporter/index.html

Otherwise, fine wit me!

Emily

Comment 12

[Roger Yang [:royang]]

Thanks!

Comment 13

[Pulsebot]

Pushed by [e-mail]:
https://hg.mozilla.org/integration/autoland/rev/9cf25eadcf55
Add JavaException annotation for determining the cause of a Java exception. r=gsvelto

Comment 14

[Bogdan Tara[:bogdan_tara | bogdant]]

https://hg.mozilla.org/mozilla-central/rev/9cf25eadcf55

Comment 15

[Roger Yang [:royang]]

Attachment: Bug 1671654 - Add bypass load URI delegate in load flags. r=jonalmeida

Comment 16

[Phabricator Automation]

Comment on attachment 9317760 [details]
Bug 1671654 - Add bypass load URI delegate in load flags. r=jonalmeida

Revision D169865 was moved to bug 1809269. Setting attachment 9317760 [details] to obsolete.