Markus Heiser

Results 640 comments of Markus Heiser

As we have not heard any response to this request so far, I assume that the feature is not wanted in the SearXNG core.

@tempname1024 thanks a lot for this PR, really great work .. :rocket: .. the CI fails on some unit test / don't spend to much time on it: our unit...

> While I'm not too sure about the privacy issues (because it would be togglable and disabled by default, so it would just be one more tool for the user/host...

> Prepare a pull request? Oh yes, please .. I can't promise we will merge but I would like to revive the topic :+1:

> I have a few questions as well. Similar questions we discussed these days on Matrix ("SearxNG public instances owners"), AFAIK @vojkovic is working on a solution .. Further I...

Thanks for your report :+1: / some additions .. `sudo -H ./utils/searxng.sh instance update` calls function `searxng.instance.update()`: https://github.com/searxng/searxng/blob/5332d3a0b8f94a7a984d22cbcfa1fb9c34cb3ff9/utils/searxng.sh#L568-L591 May git calls fail: ``` git fetch origin "$GIT_BRANCH" git reset --hard...

pyyaml is installed twice, first as a prerequisite: https://github.com/searxng/searxng/blob/1b8db63b3351f39d5eded87d4226ea3289113472/utils/searxng.sh#L509-L510 ``` pip install -U pyyaml |searxng| Requirement already satisfied: pip in /usr/local/searxng/searx-pyenv/lib/python3.9/site-packages (24.1.2) |searxng| Collecting pip ``` The following messages comes...

> I think this issue only happens with python .. but I cannot find this anywhere in the `requirements.txt` the name is `pytomlpp`: https://github.com/searxng/searxng/blob/2986681b31a60d67e0468affb257a52246220f11/requirements.txt#L18

Argh, my fail .. `tomli` is missed in the requirements.txt .. bugfix is on the way / thanks for pointing out :+1: The issue was not detected by unit tests...

FYI: issue is fixed in https://github.com/searxng/searxng/pull/3926 .. your review is welcome :) / thanks :+1: