# HG changeset patch
# User Marcin Kuzminski <marcin@rhodecode.com>
# Date 2017-12-18 23:19:05
# Node ID ca693205b93b727c2db29c6f039d1c404965cd9b
# Parent  b1d2c5d66d294ba20c7877c92ca02d2a1af76714

hooks: fix case for SSH hooks executed in certain condition with errors.

- forking/PR open

diff --git a/vcsserver/hooks.py b/vcsserver/hooks.py
--- a/vcsserver/hooks.py
+++ b/vcsserver/hooks.py
@@ -140,7 +140,9 @@ def _extras_from_ui(ui):
         if env_hook_data:
             hook_data = env_hook_data
 
-    extras = json.loads(hook_data)
+    extras = {}
+    if hook_data:
+        extras = json.loads(hook_data)
     return extras
 
 
@@ -167,7 +169,8 @@ def pre_pull(ui, repo, **kwargs):
 
 
 def pre_pull_ssh(ui, repo, **kwargs):
-    if _extras_from_ui(ui).get('SSH'):
+    extras = _extras_from_ui(ui)
+    if extras and extras.get('SSH'):
         return pre_pull(ui, repo, **kwargs)
     return 0
 
@@ -178,7 +181,8 @@ def post_pull(ui, repo, **kwargs):
 
 
 def post_pull_ssh(ui, repo, **kwargs):
-    if _extras_from_ui(ui).get('SSH'):
+    extras = _extras_from_ui(ui)
+    if extras and extras.get('SSH'):
         return post_pull(ui, repo, **kwargs)
     return 0