Cutlery/searxng
1
0
Fork 0
mirror of https://github.com/searxng/searxng.git synced 2025-11-21 14:03:10 -05:00
Code Issues Packages Projects Releases Wiki Activity

[fix] utils.js_variable_to_python - partial revert of 156d1eb8c (#5458)

Browse Source 
The JS string, whose encoding will be corrupted if all single quotes (followed
by a comma) are replaced with double quotes. Bug was introduced in PR #4573.

Here is a simple example in which the list get corrupted::

    >>> s = r"""[ 'foo\'', 'bar']"""
    >>> print(s)
    [ 'foo\'', 'bar']
    >>> print(s.replace("',", "\","))
    [ 'foo\'", 'bar']

Signed-off-by: Markus Heiser <markus.heiser@darmarit.de>
This commit is contained in:
Markus Heiser 2025-11-20 18:32:27 +01:00 committed by GitHub
parent 041f457dfa
commit 21a4622f23
No known key found for this signature in database
GPG Key ID: B5690EEEBB952194
1 changed files with 0 additions and 5 deletions
Show Stats Download Patch File Download Diff File Expand all files Collapse all files

5
searx/utils.py
View File

@ -807,11 +807,6 @@ def js_variable_to_python(js_variable: str) -> str:
s = _JS_DECIMAL_RE.sub(":0.", s) s = _JS_DECIMAL_RE.sub(":0.", s)
# replace the surogate character by colon # replace the surogate character by colon
s = s.replace(chr(1), ':') s = s.replace(chr(1), ':')
# replace single-quote followed by comma with double-quote and comma
# {"a": "\"12\"',"b": "13"}
# becomes
# {"a": "\"12\"","b": "13"}
s = s.replace("',", "\",")
# load the JSON and return the result # load the JSON and return the result
return json.loads(s) # pyright: ignore[reportAny] return json.loads(s) # pyright: ignore[reportAny]