From fb6627a9cc795fa77b7f211bd3994c661c0cfeff Mon Sep 17 00:00:00 2001 From: "Joao A. Candido Ramos" Date: Wed, 25 May 2022 19:37:34 +0200 Subject: [PATCH] Remove duplicated handling of /url result links (#769) It appears that result links beginning with '/url' were mistakenly commited with an inefficient filtering process in its place. With the way the code is structured, this less effective '/url' link filter took precedence over the previous link filter, and also caused users with the "open link in new tab" config enabled to no longer have access to that feature. Fixes #769 --- app/filter.py | 2 -- 1 file changed, 2 deletions(-) diff --git a/app/filter.py b/app/filter.py index 099d818..457fdc1 100644 --- a/app/filter.py +++ b/app/filter.py @@ -433,8 +433,6 @@ class Filter: # Internal google links (i.e. mail, maps, etc) should still # be forwarded to Google link['href'] = 'https://google.com' + q - elif link['href'].startswith('/url'): - link['href'] = q elif q.startswith('https://accounts.google.com'): # Remove Sign-in link link.decompose()