Conversation
|
Important
This repository does not receive automatic reviews because it has fewer than 10 stars. ⚙️ Run configurationConfiguration used: Organization UI Review profile: CHILL Plan: Team Run ID: Thanks for using CodeRabbit! It's free for OSS, and your support helps us grow. If you like it, consider giving us a shout-out. Comment |
![github-advanced-security[bot]](https://avatars.githubusercontent.com/in/57789?s=60&v=4){kind=small}
| return proxy_request(prov, subpath, request) | ||
| except Exception as exc: # surface upstream/adapter errors as 502 | ||
| log.exception("proxy error for %s/%s", provider, subpath) | ||
| return jsonify(error="proxy_error", detail=str(exc)), 502 |
| - **OpenAI / Anthropic** — straight HTTP reverse-proxy: rewrite host, swap in the | ||
| real key, forward with `requests`, return the response. | ||
| - **Bedrock** — re-issued through the proxy's own `boto3` client (handles SigV4 | ||
| signing + URL-encoding correctly). Only `invoke` is wired up. |
There was a problem hiding this comment.
No .converse functionality? Would we be changing all the current calls to use .invoke?
There was a problem hiding this comment.
Let me see how to add converse functionality
|
i'm thinking about how to adapt the llm proxy to the agent skill, some questions:
|
|
i added |
|
#73 Has this code integrated, may be better to review the code over there. |
5 participants
Added LLM Proxy for replacing OpenAI, Anthropic and Bedrock calls.