ttygroup
/
gertty
Code Issues Proposed changes

Merge "Match links by url instead of domain."

This commit is contained in:
Jenkins 2015-12-17 22:29:08 +00:00 committed by Gerrit Code Review
parent 861ef82a8f 4d268e5679
commit 7a041b0d19
1 changed files with 1 additions and 4 deletions
Show Stats Download Patch File Download Diff File Expand all files Collapse all files

5
gertty/app.py
View File

@ -187,7 +187,6 @@ class BackgroundBrowser(webbrowser.GenericBrowser):
class App(object):
simple_change_search = re.compile('^(\d+|I[a-fA-F0-9]{40})$')
url_change_search = re.compile('^https?://.*$')
def __init__(self, server=None, palette='default', keymap='default',
debug=False, verbose=False, disable_sync=False,
@ -478,11 +477,9 @@ class App(object):
trailing_filename_re = re.compile('.*(,[a-z]+)')
def parseInternalURL(self, url):
if not self.url_change_search.match(url):
if not url.startswith(self.config.url):
return None
result = urlparse.urlparse(url)
if result.netloc != self.config.hostname:
return None
change = patchset = filename = None
path = [x for x in result.path.split('/') if x]
if path: