Cutlery/searxng
1
0
Fork 0
mirror of https://github.com/searxng/searxng.git synced 2025-11-22 06:23:09 -05:00
Code Issues Packages Projects Releases Wiki Activity

[fix] searx/results.py - TypeError: object of type 'NoneType' has no len()

Browse Source 
In some engines, under certain circumstances, the content field can also have
the value ``None``; in these cases, a length check results in an exception::

    File "/usr/local/searxng/searx/results.py", line 360, in merge_two_main_results
        if len(other.content) > len(origin.content):
           ^^^^^^^^^^^^^^^^^^
    TypeError: object of type 'NoneType' has no len()

[1] https://github.com/searxng/searxng/issues/5250#issuecomment-3352863488

Reported-by: @scross01 [1]
Signed-off-by: Markus Heiser <markus.heiser@darmarit.de>
This commit is contained in:
Markus Heiser 2025-10-01 07:01:27 +02:00 committed by Markus Heiser
parent e16b6cb148
commit 748b521ac6
2 changed files with 3 additions and 3 deletions
Show Stats Download Patch File Download Diff File Expand all files Collapse all files

2
searx/engines/pdbe.py
View File

@ -67,7 +67,7 @@ def construct_body(result):
) )
thumbnail = pdbe_preview_url.format(pdb_id=result['pdb_id']) thumbnail = pdbe_preview_url.format(pdb_id=result['pdb_id'])
except KeyError: except KeyError:
content = None content = ""
thumbnail = None thumbnail = None
# construct url for preview image # construct url for preview image

4
searx/results.py
View File

@ -357,12 +357,12 @@ def merge_two_infoboxes(origin: LegacyResult, other: LegacyResult):
def merge_two_main_results(origin: MainResult | LegacyResult, other: MainResult | LegacyResult): def merge_two_main_results(origin: MainResult | LegacyResult, other: MainResult | LegacyResult):
"""Merges the values from ``other`` into ``origin``.""" """Merges the values from ``other`` into ``origin``."""
if len(other.content) > len(origin.content): if len(other.content or "") > len(origin.content or ""):
# use content with more text # use content with more text
origin.content = other.content origin.content = other.content
# use title with more text # use title with more text
if len(other.title) > len(origin.title): if len(other.title or "") > len(origin.title or ""):
origin.title = other.title origin.title = other.title
# merge all result's parameters not found in origin # merge all result's parameters not found in origin