Merge pull request #196 from gurneyalex/11.0-fix_travis2docker_unassi…

…gned_variable runbot_travis2docker: fix crash
OCA · Jan 8, 2020 · 8845595 · 8845595
2 parents b75f052 + bcba514
commit 8845595
Show file tree

Hide file tree

Showing 2 changed files with 3 additions and 2 deletions.
diff --git a/.travis.yml b/.travis.yml
@@ -12,6 +12,7 @@ cache:
     - $HOME/.cache/pip
 
 addons:
+  postgresql: "9.6"
   apt:
     packages:
       - expect-dev  # provides unbuffer utility

diff --git a/runbot_travis2docker/models/runbot_build.py b/runbot_travis2docker/models/runbot_build.py
@@ -185,15 +185,15 @@ def _get_ssh_keys(self):
         keys = ""
         for own_key in ['author', 'committer']:
             try:
-                login = response.get(own_key).get('login')
+                login = response.get(own_key, {}).get('login', '')
                 if login == '':
                     continue
                 ssh_rsa = self.repo_id._github('/users/%s/keys' % login)
                 keys += '\n' + '\n'.join(rsa['key'] for rsa in ssh_rsa)
             except (AttributeError, requests.RequestException) as err:
                 _logger.warning(
                     "Error fetching %s (%s): %s", own_key, login, err)
-                _logger.warning("Response received: %s", response)
+                _logger.debug("Response received: %s", response)
         return keys
 
     def _schedule(self):