Skip to content

Commit e08d33e

Browse files
committed
fix: validate SOCKS5 configuration to ensure ports differ on the same host
1 parent 1e256d5 commit e08d33e

1 file changed

Lines changed: 7 additions & 0 deletions

File tree

src/proxy_server.py

Lines changed: 7 additions & 0 deletions
Original file line numberDiff line numberDiff line change
@@ -184,6 +184,13 @@ def __init__(self, config: dict):
184184
self.socks_enabled = config.get("socks5_enabled", True)
185185
self.socks_host = config.get("socks5_host", self.host)
186186
self.socks_port = config.get("socks5_port", 1080)
187+
if self.socks_enabled and self.socks_host == self.host \
188+
and int(self.socks_port) == int(self.port):
189+
raise ValueError(
190+
f"listen_port and socks5_port must differ on the same host "
191+
f"(both set to {self.port} on {self.host}). "
192+
f"Change one of them in config.json."
193+
)
187194
self.fronter = DomainFronter(config)
188195
self.mitm = None
189196
self._cache = ResponseCache(max_mb=CACHE_MAX_MB)

0 commit comments

Comments
 (0)