Skip to content

Conversation

Xunzhuo
Copy link
Member

@Xunzhuo Xunzhuo commented Oct 8, 2025

  • Create new pkg/headers package to manage all x-* prefixed headers
  • Define constants for request, VSR decision tracking, and security headers
  • Update security headers to use x-vsr-* prefix for consistency
    • x-pii-violation -> x-vsr-pii-violation
    • x-jailbreak-blocked -> x-vsr-jailbreak-blocked
    • x-jailbreak-type -> x-vsr-jailbreak-type
    • x-jailbreak-confidence -> x-vsr-jailbreak-confidence
  • Replace all string literals with constants across codebase
  • Fix variable naming conflict in request_handler.go

BREAKING CHANGE: Security header names changed from x-* to x-vsr-* prefix. Clients consuming these headers must update to use new names.

- Create new pkg/headers package to manage all x-* prefixed headers
- Define constants for request, VSR decision tracking, and security headers
- Update security headers to use x-vsr-* prefix for consistency
  - x-pii-violation -> x-vsr-pii-violation
  - x-jailbreak-blocked -> x-vsr-jailbreak-blocked
  - x-jailbreak-type -> x-vsr-jailbreak-type
  - x-jailbreak-confidence -> x-vsr-jailbreak-confidence
- Replace all string literals with constants across codebase
- Fix variable naming conflict in request_handler.go

BREAKING CHANGE: Security header names changed from x-* to x-vsr-* prefix.
Clients consuming these headers must update to use new names.

Signed-off-by: bitliu <[email protected]>
Copy link

netlify bot commented Oct 8, 2025

Deploy Preview for vllm-semantic-router ready!

Name Link
🔨 Latest commit cb44b2a
🔍 Latest deploy log https://app.netlify.com/projects/vllm-semantic-router/deploys/68e64ba90d346700084854af
😎 Deploy Preview https://deploy-preview-362--vllm-semantic-router.netlify.app
📱 Preview on mobile
Toggle QR Code...

QR Code

Use your smartphone camera to open QR code link.

To edit notification comments on pull requests, go to your Netlify project configuration.

Copy link

github-actions bot commented Oct 8, 2025

👥 vLLM Semantic Team Notification

The following members have been identified for the changed files in this PR and have been automatically assigned:

📁 src

Owners: @rootfs, @Xunzhuo, @wangchen615
Files changed:

  • src/semantic-router/pkg/headers/headers.go
  • src/semantic-router/pkg/headers/headers_test.go
  • src/semantic-router/pkg/extproc/request_handler.go
  • src/semantic-router/pkg/extproc/response_handler.go
  • src/semantic-router/pkg/utils/http/response.go

vLLM

🎉 Thanks for your contributions!

This comment was automatically generated based on the OWNER files in the repository.

@rootfs rootfs merged commit 73ec55c into main Oct 8, 2025
9 checks passed
@Xunzhuo Xunzhuo deleted the refactor/centralize-headers branch October 10, 2025 11:43
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.

3 participants