fix: addresses null pointer when contributors API returns no author (#…

…489)
newrelic · Jun 22, 2020 · 8519971 · 8519971
1 parent dd5bcb7
commit 8519971
Show file tree

Hide file tree

Showing 2 changed files with 18 additions and 14 deletions.
diff --git a/.github/actions/sync-data/dist/index.js b/.github/actions/sync-data/dist/index.js
@@ -23058,13 +23058,15 @@ function formatStats(project, stats) {
   );
 
   const cachedContributors = Array.isArray(contributorStats)
-    ? contributorStats.map(i => ({
-        id: i.author.id,
-        login: i.author.login,
-        avatarUrl: i.author.avatar_url,
-        htmlUrl: i.author.html_url,
-        contributions: i.total
-      }))
+    ? contributorStats.map(i => {
+        return {
+          id: _.get(i, 'author.id'),
+          login: _.get(i, 'author.login'),
+          avatarUrl: _.get(i, 'author.avatar_url'),
+          htmlUrl: _.get(i, 'author.html_url'),
+          contributions: _.get(i, 'total')
+        };
+      })
     : [];
 
   const latestReleaseName = _.get(project, 'latestTag.nodes[0].name', false);

diff --git a/.github/actions/sync-data/src/stats-generator/processor.js b/.github/actions/sync-data/src/stats-generator/processor.js
@@ -58,13 +58,15 @@ function formatStats(project, stats) {
   );
 
   const cachedContributors = Array.isArray(contributorStats)
-    ? contributorStats.map(i => ({
-        id: i.author.id,
-        login: i.author.login,
-        avatarUrl: i.author.avatar_url,
-        htmlUrl: i.author.html_url,
-        contributions: i.total
-      }))
+    ? contributorStats.map(i => {
+        return {
+          id: _.get(i, 'author.id'),
+          login: _.get(i, 'author.login'),
+          avatarUrl: _.get(i, 'author.avatar_url'),
+          htmlUrl: _.get(i, 'author.html_url'),
+          contributions: _.get(i, 'total')
+        };
+      })
     : [];
 
   const latestReleaseName = _.get(project, 'latestTag.nodes[0].name', false);