Add AWS Bedrock API Key support to docs #4755
Conversation
|
![kiloconnect[bot]](https://avatars.githubusercontent.com/in/2193792?s=60&v=4){kind=small}
| 1. **AWS Access Keys (Recommended for Development):** | ||
| 1. **Bedrock API Key:** | ||
| - Create a Bedrock-specific API key in the AWS Console. This is a simple service-specific authentication method. | ||
| - See the [AWS documentation on Bedrock credentials](https://docs.aws.amazon.com/IAM/latest/UserGuide/id_credentials_bedrock.html) for instructions on creating an API key. |
There was a problem hiding this comment.
🚨 CRITICAL: Invalid AWS documentation link and non-existent feature
This documentation describes a "Bedrock API Key" authentication method that does not exist in AWS Bedrock. AWS Bedrock uses standard AWS authentication methods (IAM credentials, profiles, or roles) - there is no service-specific API key.
The linked URL (https://docs.aws.amazon.com/IAM/latest/UserGuide/id_credentials_bedrock.html) does not exist in AWS documentation.
Additionally, I found no corresponding code in the Kilo Code codebase to support a "Bedrock API Key" authentication method.
Please verify this feature exists before documenting it, or remove this section if it was added in error.
There was a problem hiding this comment.
It exists and works, and that link does indeed work.
Code Review SummaryStatus: No Issues Found | Recommendation: Merge OverviewThis PR updates the AWS Bedrock provider documentation to add a third authentication option: "Bedrock API Key". The changes include:
The documentation changes are well-structured and consistent with the existing format. Files Reviewed (1 file)
|
lambertjosh
requested review from
chrarnoldus
and removed request for
chrarnoldus
Add connection path sentence.
Adds simple initial docs for AWS Bedrock support.