diff options
| author | Markus Heiser <markus.heiser@darmarit.de> | 2021-10-06 18:24:19 +0200 |
|---|---|---|
| committer | Markus Heiser <markus.heiser@darmarit.de> | 2021-10-06 18:45:00 +0200 |
| commit | de0249ddae94cc5f320b3da7b434486ff22dc42d (patch) | |
| tree | a7b7756fe987e46978704f93bae895adcabb1975 /searx/utils.py | |
| parent | feb2d4dda575eb1501fb5a8427de364bed7bc179 (diff) | |
| download | searxng-de0249ddae94cc5f320b3da7b434486ff22dc42d.tar.gz searxng-de0249ddae94cc5f320b3da7b434486ff22dc42d.zip | |
[fix] don't mix loaded modules with imported modules (sys.modules)
The utils.load_module() function is used to load a python file (aka module) and
return the module's namespace. SearXNG uses this function to load *engines and
answerers* from arbitrary locations with arbitrary modifications. These are not
real python modules and it is not intended to mix this *engines and answerers*
with the python modules registered in sys.modules.
Closes: https://github.com/searxng/searxng/issues/312
Suggested-by: @dalf in https://github.com/searxng/searxng/issues/312
Signed-off-by: Markus Heiser <markus.heiser@darmarit.de>
Diffstat (limited to 'searx/utils.py')
| -rw-r--r-- | searx/utils.py | 4 |
1 files changed, 0 insertions, 4 deletions
diff --git a/searx/utils.py b/searx/utils.py index 22824d829..4c2c9e429 100644 --- a/searx/utils.py +++ b/searx/utils.py @@ -1,5 +1,4 @@ # -*- coding: utf-8 -*- -import sys import re import importlib @@ -427,13 +426,10 @@ def match_language(locale_code, lang_list=[], custom_aliases={}, fallback='en-US def load_module(filename, module_dir): modname = splitext(filename)[0] - if modname in sys.modules: - del sys.modules[modname] filepath = join(module_dir, filename) # and https://docs.python.org/3/library/importlib.html#importing-a-source-file-directly spec = importlib.util.spec_from_file_location(modname, filepath) module = importlib.util.module_from_spec(spec) - sys.modules[modname] = module spec.loader.exec_module(module) return module |