Merge pull request #11 from fedora-infra/feature/payload-changes

Careful with the repo owner
fedora-infra · Dec 10, 2014 · 9f2de85 · 9f2de85
2 parents bded7f0 + 3c08302
commit 9f2de85
Showing 1 changed file with 2 additions and 1 deletion.
diff --git a/github2fedmsg/views/webhooks.py b/github2fedmsg/views/webhooks.py
@@ -177,7 +177,8 @@ def build_fas_lookup(payload):
         usernames.add(payload['forkee']['owner']['login'])
 
     if 'repository' in payload:
-        usernames.add(payload['repository']['owner']['login'])
+        if 'login' in payload['repository']['owner']:
+            usernames.add(payload['repository']['owner']['login'])
 
     # Take all that, and roll it up into a dict mapping those to FAS
     mapping = {}