Skip to content
This repository has been archived by the owner on Sep 7, 2023. It is now read-only.

[mod] fix #903 : add a function is_accepted to engine. #992

Closed
wants to merge 1 commit into from

Conversation

dalf
Copy link
Contributor

@dalf dalf commented Aug 12, 2017

fix #903
search.py calls this function before creating a thread for the engine.
if the function doesn't exist, the function is .... not called.

without this requests a useless thread is created for the dictzone and currency engines.

@dalf
Copy link
Contributor Author

dalf commented Aug 12, 2017

Another more simple implementation : just define a regex for an engine.
If search.py see a defined regex, it checks if the query matches this regex.

dictzone and translated do extended check of the languages in is_accepted, with the regex version it won't be possible.

… calls this function creating a thread for the engine.
@dalf dalf added the core label May 12, 2018
@asciimoo asciimoo added this to Milestone #2 - Backend in v1.0.0 Mar 4, 2020
searx/search.py Show resolved Hide resolved
@dalf
Copy link
Contributor Author

dalf commented Sep 26, 2020

Supersed by #2159

@dalf dalf closed this Sep 26, 2020
v1.0.0 automation moved this from Milestone #2 - Backend to Done Sep 26, 2020
@dalf dalf deleted the is_accepted branch February 21, 2021 20:18
Sign up for free to subscribe to this conversation on GitHub. Already have an account? Sign in.
Labels
Projects
No open projects
v1.0.0
  
Done
Development

Successfully merging this pull request may close these issues.

An useless thread is created on each request in the general category
1 participant