github/AutoGPT
1
1
Fork 0
mirror of https://github.com/Significant-Gravitas/AutoGPT.git synced 2026-04-30 03:00:41 -04:00
Code Issues Packages Projects Releases Wiki Activity

fix(copilot/sdk-proxy): treat empty sdk_env ANTHROPIC_BASE_URL as opt-out

Browse Source 
Claude Code subscription mode intentionally sets
``sdk_env['ANTHROPIC_BASE_URL'] = ""`` to disable any base-URL
override and keep the CLI talking to Anthropic directly. The
previous ``or``-chained lookup evaluated the empty string as falsy
and fell through to ``os.environ.get("ANTHROPIC_BASE_URL")`` and
then to ``OPENROUTER_BASE_URL``, silently starting the compat proxy
for a session that had explicitly opted out — which breaks
subscription auth.

Use a presence check on ``sdk_env`` instead: if the key is present
with an empty value it's a hard "no-proxy" signal, so skip the
OpenRouter fallback even when ``openrouter_active`` is True. The
process-env fallback and the OpenRouter fallback still cover the
original cases (no sdk_env override, OpenRouter is the routing
provider for this session).

Flagged by sentry review on #12745 (thread 3067906804).
This commit is contained in:
majdyz
2026-04-11 10:48:04 +00:00
parent 370499c8dc
commit 8742c5e5b9
1 changed files with 28 additions and 4 deletions
Download Patch File Download Diff File Expand all files Collapse all files

32
autogpt_platform/backend/backend/copilot/sdk/service.py
View File

@@ -2277,10 +2277,34 @@ async def stream_chat_completion_sdk(
# feature is opt-in and documented as "OpenRouter
# compatibility", so quietly no-oping on direct-Anthropic
# sessions is the safe default.
target_base_url: str | None = (sdk_env or {}).get(
"ANTHROPIC_BASE_URL"
) or os.environ.get("ANTHROPIC_BASE_URL")
if not target_base_url and config.openrouter_active:
# Claude Code subscription mode intentionally sets
# ``sdk_env['ANTHROPIC_BASE_URL'] = ""`` to *disable* any
# base-URL override and keep the CLI talking to Anthropic
# directly. Treat an explicit empty string as a hard
# "no-proxy" signal so we never silently start the proxy
# against a host-wide ``ANTHROPIC_BASE_URL`` or fall back
# to OpenRouter when the caller has opted out.
sdk_env_map = sdk_env or {}
explicit_sdk_env = "ANTHROPIC_BASE_URL" in sdk_env_map
sdk_env_value = (
sdk_env_map["ANTHROPIC_BASE_URL"] if explicit_sdk_env else None
)
if explicit_sdk_env and not sdk_env_value:
# Empty string from sdk_env → subscription mode opt-out.
target_base_url: str | None = None
explicit_opt_out = True
else:
target_base_url = sdk_env_value or os.environ.get("ANTHROPIC_BASE_URL")
explicit_opt_out = False
# Only fall back to OpenRouter when the session actually
# has no base-URL plumbing of its own AND OpenRouter is
# the active routing provider AND the caller hasn't
# explicitly opted out via an empty sdk_env override.
if (
not target_base_url
and not explicit_opt_out
and config.openrouter_active
):
from backend.util.clients import OPENROUTER_BASE_URL
target_base_url = OPENROUTER_BASE_URL