这是indexloc提供的服务,不要输入任何密码
Skip to content

Conversation

@return42
Copy link
Member

@return42 return42 commented Nov 1, 2023

In PR-2894[1] we isolated botdetection from the limiter, this PR isolates the botdetection from the SearXNG core code.

This PR also fixes the issue [2] that the server.public_instance option needs to activate the limiter.

Resources of botdetection

Package botdetection at PyPi:

Repository of botdetection at SearXNG org:

In PR-2894[1] we isolated botdetection from the limiter, this PR isolates the
botdetection from the SearXNG core code.

This PR also fixes the issue [2] that the ``server.public_instance`` option
needs to activate the limiter.

- [1] searxng#2894
- [2] searxng#2975

Signed-off-by: Markus Heiser <markus.heiser@darmarit.de>
Signed-off-by: Markus Heiser <markus.heiser@darmarit.de>
@return42 return42 force-pushed the public_instance-settings branch from a29b963 to a2ee107 Compare February 26, 2024 16:25
@unixfox
Copy link
Contributor

unixfox commented Feb 26, 2024

Not sure if something is going to change about the filenames but if so then we will need to synchronize them with the one located in searxng-docker: https://github.com/searxng/searxng-docker/blob/master/searxng/limiter.toml

DEFAULT_CFG = Path(__file__).parent / "limiter.toml"
"""Base configuration (schema) of the botdetection."""

LIMITER_CFG = Path('/etc/searxng/limiter.toml')
Copy link
Member Author

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

Not sure if something is going to change about the filenames but if so then we will need to synchronize them with the one located in searxng-docker: https://github.com/searxng/searxng-docker/blob/master/searxng/limiter.toml

The LIMITER_CFG kept unchanged.

@dalf
Copy link
Contributor

dalf commented Mar 2, 2024

I've started a version with a Flask middleware (prototype not even run, just a bunch of code to explain how the botdetection and searxng can be decoupled):

[EDIT] the code and the example work now, however the documentation and comments are not updated.

@dalf
Copy link
Contributor

dalf commented Mar 3, 2024

New update:

Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Labels

None yet

Projects

None yet

Development

Successfully merging this pull request may close these issues.

3 participants