fix(models): use x-api-key for Anthropic API probing#13189
Closed
cedric-common wants to merge 1 commit into
Closed
fix(models): use x-api-key for Anthropic API probing#13189cedric-common wants to merge 1 commit into
cedric-common wants to merge 1 commit into
Conversation
probe_api_models() unconditionally sends Authorization: Bearer to every endpoint. Anthropic's native API (api.anthropic.com) rejects Bearer tokens and requires x-api-key + anthropic-version headers. This was previously masked because detect_provider_for_model() silently rerouted Anthropic requests to OpenRouter (fixed in f2f9d0c). Now that direct provider routing is respected, the probing mismatch surfaces as a 401 and blocks native Anthropic usage. Fix: detect api.anthropic.com in probe_api_models() and swap the auth headers before making the /models request.
✅ Review Complete - LGTMTested - all 49 Anthropic provider tests passing Test Results
Detailed AnalysisAnthropic x-api-key fix ✅
Test coverage ✅
Impact ✅
RecommendationMerge immediately. This fix:
No additional changes needed. Ready for merge. Reviewer: @teknium1 |
Collaborator
1 similar comment
Collaborator
Contributor
|
Thanks @cedric-common. Closing — subsumed by @Wangshengyang2004's #12950 which landed in #15136 (commit 647900e). Your URL-based detection ( Both contributors identified the right problem — thanks for the report. |
This file contains hidden or bidirectional Unicode text that may be interpreted or compiled differently than what appears below. To review, open the file in an editor that reveals hidden Unicode characters.
Learn more about bidirectional Unicode characters
Sign up for free
to join this conversation on GitHub.
Already have an account?
Sign in to comment
4 participants
Add this suggestion to a batch that can be applied as a single commit.This suggestion is invalid because no changes were made to the code.Suggestions cannot be applied while the pull request is closed.Suggestions cannot be applied while viewing a subset of changes.Only one suggestion per line can be applied in a batch.Add this suggestion to a batch that can be applied as a single commit.Applying suggestions on deleted lines is not supported.You must change the existing code in this line in order to create a valid suggestion.Outdated suggestions cannot be applied.This suggestion has been applied or marked resolved.Suggestions cannot be applied from pending reviews.Suggestions cannot be applied on multi-line comments.Suggestions cannot be applied while the pull request is queued to merge.Suggestion cannot be applied right now. Please check back later.
Problem
🐛 Describe the bug
detect_provider_for_model() silently rerouted Anthropic requests to OpenRouter (fixed in #10300 / f2f9d0c). Now that direct provider routing is respected, unconditionally sends to every endpoint. Anthropic's native API () rejects Bearer tokens and returns HTTP 401, which blocks native Anthropic usage entirely.
Reproduction
Fix
Detect in and swap the auth headers before making the request:
Checklist
Closes the latent probing issue exposed by #10300.