Add other backend authentication type to LLM Provider#2706
Conversation
|
Warning Review limit reached
Next review available in: 54 minutes Enable usage-based reviews in Billing to review now. Otherwise, wait until the next included review is available. How can I continue?After more reviews become available, a review can be triggered using the To avoid repeated limits, reduce automatic review volume by pausing incremental auto-reviews earlier, using label-based review opt-in, excluding WIP or generated PR titles, or requesting reviews manually when the PR is ready. If your team needs uninterrupted high-volume reviews, an organization admin can enable usage-based reviews. How do review limits work?CodeRabbit enforces per-developer PR review limits for each organization. Most developers receive the normal plan review availability. For paid Pro and Pro+ PR reviews, CodeRabbit uses adaptive limits for sustained high-volume activity. When a developer's recent PR review activity reaches the 95th percentile or higher among CodeRabbit users, additional reviews become available more gradually as earlier reviews age out of the rolling window. Please refer docs for additional details. Review details⚙️ Run configurationConfiguration used: Path: .coderabbit.yaml Review profile: CHILL Plan: Pro Run ID: 📒 Files selected for processing (9)
✨ Finishing Touches🧪 Generate unit tests (beta)
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 |
|
Caution Failed to replace (edit) comment. This is likely due to insufficient permissions or the comment being deleted. Error details |
|
@coderabbitai please review |
|
The PR description image shows the "Add Provider" form in the LLM Providers UI with an Anthropic provider template, where "Authentication Type" is set to "other" and a helper message reads: "No credentials are stored for this provider. Use a policy to configure authentication." The second PR description image shows the provider detail page for "BedrockTest" (AWS Bedrock), on the "Connection" tab, confirming the created provider has "Authentication" set to "other" with the same helper message, and a 3/4 setup progress indicator showing "Add Guardrails", "Deploy to Gateway" (done), and "Consume LLM Provider" (done).
✅ Action performedReview finished.
|
Purpose
This pull request adds support for a new upstream authentication type,
"other", which allows authentication to be handled entirely by user-attached policies rather than by storing credentials in the platform. The changes ensure that when"other"is selected, credential fields are hidden in the UI, no credentials are stored, and the deployment artifact omits the auth block. Comprehensive backend and frontend updates, as well as tests, are included to support this new authentication flow.Backend changes:
"other"as a validUpstreamAuthTypethroughout the API, OpenAPI spec, and normalization logic. [1] [2] [3] [4]authblock if the type is"other", preventing the gateway from attaching header-setting policies. [1] [2]authblock is omitted for"other"in both provider and proxy deployment YAML.Frontend/UI changes:
"other"as an authentication type, hide credential fields, and display a helper message when selected. [1] [2] [3] [4] [5] [6] [7] [8] [9] [10] [11] [12] [13] [14]"other"clears any previously entered credential values in the form state. [1] [2]Type definition updates:
UpstreamAuthTypeScript interface to include"other"as a valid type.