Merge pull request #2742 from chaoss/main-facade-hotfix

Facade Quick Fix
chaoss · Mar 15, 2024 · 9d69d7b · 9d69d7b
2 parents 46d396d + 0557151
commit 9d69d7b
Show file tree

Hide file tree

Showing 2 changed files with 7 additions and 2 deletions.
diff --git a/augur/tasks/git/facade_tasks.py b/augur/tasks/git/facade_tasks.py
@@ -252,7 +252,7 @@ def analyze_commits_in_parallel(repo_git, multithreaded: bool)-> None:
         session.log_activity('Debug',f"Commits missing from repo {repo_id}: {len(missing_commits)}")
 
 
-        if not len(missing_commits):
+        if not len(missing_commits) or repo_id is None:
             #session.log_activity('Info','Type of missing_commits: %s' % type(missing_commits))
             return
 

diff --git a/augur/tasks/github/facade_github/contributor_interfaceable/contributor_interface.py b/augur/tasks/github/facade_github/contributor_interfaceable/contributor_interface.py
@@ -367,7 +367,12 @@ def get_login_with_supplemental_data(logger,db,auth, commit_data):
         return None
 
     # Grab first result and make sure it has the highest match score
-    match = login_json['items'][0]
+    try:
+        match = login_json['items'][0]
+    except IndexError as e:
+        logger.error(f"Ran into error {e} when parsing users with search url: {url}\n return dict: {login_json}")
+        return None
+
     for item in login_json['items']:
         if item['score'] > match['score']:
             match = item