Skip to content

Navigation Menu

Sign in
Appearance settings

Search code, repositories, users, issues, pull requests...

Provide feedback

We read every piece of feedback, and take your input very seriously.

Saved searches

Use saved searches to filter your results more quickly

Appearance settings

Fix Redis self-signed TLS and Bedrock compat routing - #2420

#2420
Open
mr-kelly wants to merge 2 commits into
Wei-Shaw:mainWei-Shaw/sub2api:mainfrom
vikadata:mainvikadata/sub2api:mainCopy head branch name to clipboard
Open

Fix Redis self-signed TLS and Bedrock compat routing#2420
mr-kelly wants to merge 2 commits into
Wei-Shaw:mainWei-Shaw/sub2api:mainfrom
vikadata:mainvikadata/sub2api:mainCopy head branch name to clipboard

Conversation

@mr-kelly

Copy link
Copy Markdown

Summary

This PR includes two focused fixes that should be generally useful upstream:

  1. Support Redis TLS deployments that use private or self-signed certificates.
  2. Route Bedrock compatibility requests through the Bedrock runtime path instead of the generic Anthropic/OpenAI forwarders.

Changes

Redis TLS

  • Add redis.tls_skip_verify configuration support.
  • Wire the option into the Redis TLS client config.
  • Expose the setting in setup and deployment examples.
  • Update Redis setup/config tests for the new field.

This is useful for private deployments where Redis is served over TLS with an internal CA or a self-signed certificate.

Bedrock compatibility routing

  • Add Bedrock compatibility request handling for chat completions and responses forwarding.
  • Preserve Bedrock-specific model/provider routing when requests enter through compatibility endpoints.
  • Add tests covering Bedrock compat request behavior.

This prevents Bedrock accounts from being incorrectly routed through non-Bedrock upstream paths when using compat endpoints.

Tests

go test ./internal/repository ./internal/service -run 'Test.*Redis|Test.*Bedrock|Bedrock|Redis'
git diff --check upstream/main..HEAD

paylm added 2 commits May 13, 2026 07:14
(cherry picked from commit 59988c03ded3b6753fdc2a0cf88971ac1756ea9c)
* fix: route bedrock compat endpoints through bedrock runtime

* fix: resolve bedrock compat build errors

(cherry picked from commit cc37a4c4dc2c5fa863b0dd88088cc1dd6d46c616)
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Labels

None yet

Projects

None yet

Development

Successfully merging this pull request may close these issues.

2 participants

Morty Proxy This is a proxified and sanitized view of the page, visit original site.