opensource/AstrBot
3
0
Fork 0
mirror of https://github.com/AstrBotDevs/AstrBot.git synced 2026-09-24 16:39:52 +08:00
Code Issues Packages Projects Releases Wiki Activity

fix(proxy): ensure AstrBot ignores system proxy variables when no proxy is configured to prevent local API requests from being intercepted (#8897)

Browse Source 
closes #8871

---------

Co-authored-by: Soulter <905617992@qq.com>
This commit is contained in:
NayukiChiba
2026-07-28 21:52:08 +08:00
committed by GitHub
co-authored by Soulter
parent ad22c676b6
commit d9790e3ea6
2 changed files with 15 additions and 2 deletions
Download Patch File Download Diff File Expand all files Collapse all files
astrbot/core/core_lifecycle.py
+13 -2
View File
@@ -74,14 +74,25 @@ class AstrBotCoreLifecycle:
no_proxy_list = self.astrbot_config.get("no_proxy", [])
os.environ["no_proxy"] = ",".join(no_proxy_list)
else:
# 清空代理环境变量
# Clear system proxy variables to avoid interfering with localhost requests.
has_system_proxy = "https_proxy" in os.environ or "http_proxy" in os.environ
if has_system_proxy:
logger.warning(
"System http_proxy/https_proxy environment variables were detected, "
"but AstrBot has no proxy configured. Clearing the proxy variables "
"and setting no_proxy to localhost,127.0.0.1,::1 so local API "
"requests bypass the proxy. Configure http_proxy in AstrBot if a "
"proxy is required."
)
if "https_proxy" in os.environ:
del os.environ["https_proxy"]
if "http_proxy" in os.environ:
del os.environ["http_proxy"]
if "no_proxy" in os.environ:
del os.environ["no_proxy"]
logger.debug("HTTP proxy cleared")
# Always bypass proxies for loopback addresses used by local APIs.
os.environ["no_proxy"] = "localhost,127.0.0.1,::1"
logger.debug("HTTP proxy cleared, no_proxy set to localhost")
async def _init_or_reload_subagent_orchestrator(self) -> None:
"""Create (if needed) and reload the subagent orchestrator from config.
tests/unit/test_core_lifecycle.py
+2
View File
@@ -98,6 +98,8 @@ class TestAstrBotCoreLifecycleInit:
# Verify proxy environment variables are cleared
assert "http_proxy" not in os.environ
assert "https_proxy" not in os.environ
# Verify local APIs always bypass proxies after clearing the environment.
assert os.environ.get("no_proxy") == "localhost,127.0.0.1,::1"
class TestAstrBotCoreLifecycleStop: