Improvement suggestions

demostanis / gimmeasearx

Find a random searx instance

http://7tcuoi57curagdk7nsvmzedcxgwlrq2d6jach4ksa3vj72uxrzadmqqd.onion/

Other

47 stars 7 forks source link

Improvement suggestions #6

Open ghost opened 3 years ago

ghost commented 3 years ago

Hey, I've been using your tool for a few weeks and I have a lot to thank you for. It really is something very useful! I have some suggestions that can help improve the project, especially for those who used searx.neocities.org before.

Filter by version More than half of the instances available today are in an old version. So the chance that one of them will be used is very high.
Exclusion list for countries At least for the US 🤣
List of allowed instances It is currently possible to use an exclusion list, but the instances are very dynamic, so excluding some does not guarantee that if a new instance is not used
Delete doubtful instances Searx.neocities.org uses the following filter

network: Does not contain "googleusercontent" http headers: Does not contain "CloudFlare" or "googleusercontent"
TLS and CSP This is as important as the filter that currently exists for HTML and JS

demostanis commented 3 years ago

Hey, thanks for your suggestions and sorry for the late reply. I will try to implement them in the following days!

demostanis commented 3 years ago

About the allowlist, should it completly replace the blacklist or should there be an option to toggle between them?

demostanis commented 3 years ago

Wait did he delete his account?

fabianski7 commented 3 years ago

Wait did he delete his account?

it seems so

I think each one can be useful for many different users. maybe an option to select between both?

Personally, what bothers me most is using instances with old versions

demostanis commented 3 years ago

Added support for filtering by version in master. For the exclusion list by country, I am not sure how this could be implemented in our UI, as it mustn't use JavaScript. Should it list flags with checkboxes for the ones to keep? It would also depend on an API to fetch searx instances' IP addresses information, which may get the public instance rate limited after some time.

demostanis commented 3 years ago

Removing of instances using Cloudflare is in master too. I don't think it should be an option, who the fuck wants instances using Cloudflare? Imma go play Red Dead Redemption 2.

fabianski7 commented 3 years ago

everything seems to be working. thanks! enjoy the game

demostanis commented 3 years ago

List of allowed instances can be done by blacklisting * (everything) and putting wanted instances in Custom instances. It might be a little harder to filter by country as the UI cannot use JS, and that knowing where the instance is located would require dependence to an API providing this information (how would it be possible otherwise?). I'd need opinion of whomever's reading this. For TLS and CSP, it is in my list of things to be done. Pull requests are much appreciated.

MrGeorgen commented 3 years ago

List of allowed instances can be done by blacklisting * (everything) and putting wanted instances in Custom instances. It might be a little harder to filter by country as the UI cannot use JS, and that knowing where the instance is located would require dependence to an API providing this information (how would it be possible otherwise?). I'd need opinion of whomever's reading this. For TLS and CSP, it is in my list of things to be done. Pull requests are much appreciated.

if I put * into the blacklist and instances into custom instances and save it, the custom instances are just reset. If manually add the instances into the url they are still ignored

demostanis commented 3 years ago

Oops, my bad. You need to blacklist .*, as the blacklist is composed of regexes.