search

jxsl13 / TeeworldsEconVPNDetection

The Go implementation of an external VPN ban tool that connects to your server via the econ(based on telnet)
MIT License
2 stars 1 forks source link

make proxy detection optional, toggle able #5

Closed jxsl13 closed 3 years ago

jxsl13 commented 3 years ago

chillerdraghuhn:

i mean home servers are a thing so it might have false positives by default
ChillerDragon commented 3 years ago

Imo even turned on it could try to reduce false positives. If one sets up a test server using the default name "unnamed server" it is likley not a malicious proxy server trying to steal data.

Another idea is to only flag those servers as proxy that duplicate an existing server by same player count or same name (should probably also detect confusable names).

jxsl13 commented 3 years ago

I wonder how many IPs overlap when fetching the teeworlds 0.7 server list vs the ddnet http master server. fetching the individual server information for every ip returned by the master server is a pain in the butt :D

jxsl13 commented 3 years ago

in fact, the http json master servers contains all 0.6 and all 0.7 servers, making the usage of my twapi library obsolete.

jxsl13 commented 3 years ago

this allows for some nice improvements in this issue

jxsl13 commented 3 years ago

dc3e1c99ce6f44a7a82ba30a8f7f4bba57af1b36 implements this server name distance check which decreases the list of cached IPs from like 200 to anything you like. the current default value 8 decreases the list of servers similar to my zCatch/gCTF servers from 200 to 3 IPs and 5 servers.