report: add report versioning

cjihrig · targos · commit c4a357dadaff · 2019-07-02T09:07:44.000+02:00
This commit adds a version to the diagnostic report feature. PR-URL: #28121 Reviewed-By: Anna Henningsen <anna@addaleax.net> Reviewed-By: Gireesh Punathil <gpunathi@in.ibm.com>
diff --git a/doc/api/report.md b/doc/api/report.md
@@ -22,6 +22,7 @@ is provided below for reference.
 ```json
 {
   "header": {
+    "reportVersion": 1,
     "event": "exception",
     "trigger": "Exception",
     "filename": "report.20181221.005011.8974.0.001.json",
diff --git a/src/node_report.cc b/src/node_report.cc
@@ -24,6 +24,7 @@
 extern char** environ;
 #endif
 
+constexpr int NODE_REPORT_VERSION = 1;
 constexpr int NANOS_PER_SEC = 1000 * 1000 * 1000;
 constexpr double SEC_PER_MICROS = 1e-6;
 
@@ -172,7 +173,7 @@ static void WriteNodeReport(Isolate* isolate,
   JSONWriter writer(out);
   writer.json_start();
   writer.json_objectstart("header");
-
+  writer.json_keyvalue("reportVersion", NODE_REPORT_VERSION);
   writer.json_keyvalue("event", message);
   writer.json_keyvalue("trigger", trigger);
   if (!filename.empty())
diff --git a/test/common/report.js b/test/common/report.js
@@ -63,8 +63,10 @@ function _validateContent(data) {
                         'nodejsVersion', 'wordSize', 'arch', 'platform',
                         'componentVersions', 'release', 'osName', 'osRelease',
                         'osVersion', 'osMachine', 'cpus', 'host',
-                        'glibcVersionRuntime', 'glibcVersionCompiler', 'cwd'];
+                        'glibcVersionRuntime', 'glibcVersionCompiler', 'cwd',
+                        'reportVersion'];
   checkForUnknownFields(header, headerFields);
+  assert.strictEqual(header.reportVersion, 1);  // Increment as needed.
   assert.strictEqual(typeof header.event, 'string');
   assert.strictEqual(typeof header.trigger, 'string');
   assert(typeof header.filename === 'string' || header.filename === null);

Original file line number	Diff line number	Diff line change
`@@ -22,6 +22,7 @@ is provided below for reference.`
`22`	`22`	```json
`23`	`23`	`{`
`24`	`24`	`"header": {`
	`25`	`+ "reportVersion": 1,`
`25`	`26`	`"event": "exception",`
`26`	`27`	`"trigger": "Exception",`
`27`	`28`	`"filename": "report.20181221.005011.8974.0.001.json",`