From 6d7e86eece852a525ac2ec3c191aca57712ac1a7 Mon Sep 17 00:00:00 2001 From: Markus Heiser Date: Tue, 11 Jan 2022 08:55:10 +0100 Subject: [PATCH] [fix] get_bang_url: handle ambiguous !!bangs without error An ambiguous bang like `!!d` raises an exception in function get_bang_url(). A bang is only unique when the bang_definition from get_bang_definition_and_ac() is a string / for a ambiguous bang the returned bang_definition is a dictionary. Reported-by: user prg at #searxng:matrix.org on 2022/01/11 Signed-off-by: Markus Heiser --- searx/external_bang.py | 7 +++++-- 1 file changed, 5 insertions(+), 2 deletions(-) diff --git a/searx/external_bang.py b/searx/external_bang.py index f8e872693..ec5a46ed0 100644 --- a/searx/external_bang.py +++ b/searx/external_bang.py @@ -77,11 +77,14 @@ def get_bang_url(search_query, external_bangs_db=None): :param search_query: This is a search_query object which contains preferences and the submitted queries. :return: None if the bang was invalid, else a string of the redirect url. """ + ret_val = None + if external_bangs_db is None: external_bangs_db = EXTERNAL_BANGS if search_query.external_bang: bang_definition, _ = get_bang_definition_and_ac(external_bangs_db, search_query.external_bang) - return resolve_bang_definition(bang_definition, search_query.query)[0] if bang_definition else None + if bang_definition and isinstance(bang_definition, str): + ret_val = resolve_bang_definition(bang_definition, search_query.query)[0] - return None + return ret_val