louislam / uptime-kuma

A fancy self-hosted monitoring tool
https://uptime.kuma.pet
MIT License
59.87k stars 5.36k forks source link

Add proxying support for TCP Port (SOCKS v5) #1949

Open gi-yt opened 2 years ago

gi-yt commented 2 years ago

⚠️ Please verify that this feature request has NOT been suggested before.

🏷️ Feature Request Type

Other

🔖 Feature description

Even if socks5 is used for proxying, proxying support is only available for the http(s) mode and not TCP Port/Ping modes... This shouldn't be too hard to implement since SOCKS v5 does support this unlike HTTP proxies..

✔️ Solution

Implement proxying for SOCKS based proxies on TCP Port/Ping mode

❓ Alternatives

N/A

📝 Additional Context

image With HTTP(s) mode ^ image With TCP Port mode ^

chakflying commented 2 years ago

FYI since the tcp-ping library and node.js's Socket.connect() does not support the use of proxies, this would not be very easy. We may have to switch to another library like socks, and possibly rewrite or modify tcp-ping to make use of it.

Ran-Xing commented 1 year ago

I think this is more important, after all, I don't want to deploy multiple uptimes at the same time

In some important network environments, if you can open fewer ports, open fewer ports

ShadowVoyd commented 1 year ago

I would like to see this implemented as well.