# HG changeset patch
# User Marcin Kuzminski <marcin@rhodecode.com>
# Date 2017-12-18 14:12:41
# Node ID b1d2c5d66d294ba20c7877c92ca02d2a1af76714
# Parent  a3726eeb7227b0cdb6c42916fad34f4cea77474d

git: use True/False on check git repo path call. This prevents spamming
rhodecode logs with fake exceptions about missing GitRepos which are meaningless
and confuses customers.

diff --git a/vcsserver/git.py b/vcsserver/git.py
--- a/vcsserver/git.py
+++ b/vcsserver/git.py
@@ -20,6 +20,7 @@ import os
 import posixpath as vcspath
 import re
 import stat
+import traceback
 import urllib
 import urllib2
 from functools import wraps
@@ -124,11 +125,15 @@ class GitRemote(object):
 
     @reraise_safe_exceptions
     def assert_correct_path(self, wire):
+        path = wire.get('path')
         try:
             self._factory.repo(wire)
         except NotGitRepository as e:
-            # Exception can contain unicode which we convert
-            raise exceptions.AbortException(repr(e))
+            tb = traceback.format_exc()
+            log.debug("Invalid Git path `%s`, tb: %s", path, tb)
+            return False
+
+        return True
 
     @reraise_safe_exceptions
     def bare(self, wire):