[mod] Add new instances template: use GitHub forms

searx / searx-instances

Searx instance list

https://searx.space/

GNU Lesser General Public License v3.0

91 stars 19 forks source link

[mod] Add new instances template: use GitHub forms #189

Closed dalf closed 3 years ago

dalf commented 3 years ago

related to #188

About antibot section:

New fields:

country (? searx.space detect this field)
MITM (searx.space can detect some, but the list is not maintained: https://github.com/searxng/searx-space/blob/master/searxstats/data/asn.py )
source code:
- /config endpoint provides git_url
- in searx, the value comes settings.yml:brand.git_url, which might be not updated.
- here this field explicitly ask to provide the URL
- it is possible to specify the git_url in instances.yml

return42 commented 3 years ago

Is this template what we can see in your master branch?

https://github.com/dalf/searx-instances/issues/new?assignees=&labels=instance+add%2Cinstance&template=add-instance.yaml&title=Add+%3Cinstance+url%3E

dalf commented 3 years ago

Yes.

[EDIT] I'm going to do some experiments, so the form might be sightly different.

dalf commented 3 years ago

@return42 I've updated the form.

ping @unixfox : see the form on dalf/searx-instances or the source code here.

unixfox commented 3 years ago

"Man in the Middle" section is not that useful because searx.space already flag the networks that are doing some MITM like cloudflare. This is useful for invidious because there is no such check.

I would suggest to remove this part.

return42 commented 3 years ago

I would suggest to remove this part.

it is not in, was just an suggestion of mine .. but you are right, we tag them already :

grafik

unixfox commented 3 years ago

I would suggest to remove this part.

it is not in, was just an suggestion of mine .. but you are right, we tag them already :

Hum yes it is in: https://github.com/searx/searx-instances/pull/189/files#diff-e96f9e68cb96cce4c8d0448962f8026d7cfeec7725decd8b74456fa1e004c26eR41

return42 commented 3 years ago

Argh, sorry I mixed up things .. it is in!

Sorry for the mess ..

dalf commented 3 years ago

Removed.

See the form on dalf/searx-instances or the source code here.

unixfox commented 3 years ago

I've added more changes, @dalf feel free to squash all the commits into one (I don't have git cli installed right now).

return42 commented 3 years ago

What about maintenance .. should we add a section about (see my comment above)?

unixfox commented 3 years ago

What about maintenance .. should we add a section about (see my comment above)?

I'm going to merge this PR but feel free to submit another one for the "maintenance" section.

Ultimately, searx.space could track the uptime of every searx instance and rank them automatically so that we don't stipulate anything about "required" uptime but that's still a lot of work to develop a feature like this.