fix(sdk): inject API key into all HTTP requests #50
Conversation
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
The Python SDK was not including the X-API-Key header in HTTP requests made through AgentFieldClient._async_request(), causing 401 errors when the control plane has authentication enabled. This fix injects the API key into request headers automatically when: - The client has an api_key configured - The header isn't already set (avoids overwriting explicit headers) Fixes async status updates and memory operations (vector search, etc.) that were failing with 401 Unauthorized. 🤖 Generated with [Claude Code](https://claude.com/claude-code) Co-Authored-By: Claude <noreply@anthropic.com>
Sign up for free
to join this conversation on GitHub.
Already have an account?
Sign in to comment
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.
Summary
X-API-Keyheader inAgentFieldClient._async_request()whenapi_keyis configuredProblem
When authentication is enabled on the control plane, the Python SDK was failing with 401 errors on:
_post_execution_status)The SDK had
api_keyconfigured but wasn't including it in theX-API-Keyheader for HTTP requests.Solution
Central fix at
AgentFieldClient._async_request()that automatically injects the API key into all outbound requests. This covers both async (httpx) and sync fallback (requests) paths.Test plan
🤖 Generated with Claude Code