Skip to content

fix(volcengine): forward model request headers#2909

Merged
qin-ctx merged 1 commit into
mainfrom
fix/volcengine-extra-headers
Jul 1, 2026
Merged

fix(volcengine): forward model request headers#2909
qin-ctx merged 1 commit into
mainfrom
fix/volcengine-extra-headers

Conversation

@qin-ctx

@qin-ctx qin-ctx commented Jun 30, 2026

Copy link
Copy Markdown
Collaborator

Description

Add Volcengine model request header forwarding for VLM and embedding calls. Volcengine requests now default X-Client-Request-Id to ToB-direct,OpenViking_Service,openviking-service_cn-beijing while preserving caller-provided extra_headers overrides.

Related Issue

N/A

Type of Change

  • Bug fix (non-breaking change that fixes an issue)
  • New feature (non-breaking change that adds functionality)
  • Breaking change (fix or feature that would cause existing functionality to not work as expected)
  • Documentation update
  • Refactoring (no functional changes)
  • Performance improvement
  • Test update

Changes Made

  • Forward Volcengine VLM extra_headers through chat completion requests, including bot streaming calls.
  • Add the default Volcengine X-Client-Request-Id header while allowing explicit header overrides.
  • Forward embedding extra_headers for Volcengine dense, sparse, and hybrid embedding calls via existing embedding config.

Testing

  • I have added tests that prove my fix is effective or that my feature works
  • New and existing unit tests pass locally with my changes
  • I have tested this on the following platforms:
    • Linux
    • macOS
    • Windows

Manual checks:

  • Ran python -m compileall -q on the changed implementation files.
  • Captured local HTTP requests for chat completions, text embeddings, and multimodal embeddings to confirm X-Client-Request-Id and configured extra headers are sent as request headers and not in the JSON body.

Checklist

  • My code follows the project's coding style
  • I have performed a self-review of my code
  • I have commented my code, particularly in hard-to-understand areas
  • I have made corresponding changes to the documentation
  • My changes generate no new warnings
  • Any dependent changes have been merged and published

Screenshots (if applicable)

N/A

Additional Notes

No tests were added per request.

Ensure Volcengine VLM and embedding calls propagate configured headers and default the client request id header for service traffic.
@qin-ctx qin-ctx force-pushed the fix/volcengine-extra-headers branch from 36eda30 to b09671a Compare July 1, 2026 06:30
@qin-ctx qin-ctx merged commit 0770822 into main Jul 1, 2026
3 checks passed
@qin-ctx qin-ctx deleted the fix/volcengine-extra-headers branch July 1, 2026 06:31
@github-project-automation github-project-automation Bot moved this from Backlog to Done in OpenViking project Jul 1, 2026
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Labels

None yet

Projects

Status: Done

Development

Successfully merging this pull request may close these issues.

2 participants