Skip to content

Conversation

@Bobholamovic
Copy link
Contributor

No description provided.

@Bobholamovic Bobholamovic changed the title feat: Add 'Client-Platform' header for server processing feat: Add 'Client-Platform' header for server processing for PP-OCR and PP-Structure Jan 7, 2026
@fanlv
Copy link
Collaborator

fanlv commented Jan 7, 2026

Why do we need to add this field?

@Bobholamovic
Copy link
Contributor Author

Why do we need to add this field?

This is a new specification introduced for the official PaddleOCR service. By identifying the source of client requests, we will be able to implement additional server-side extensions in the future, such as providing special handling for certain clients or enabling more fine-grained traffic control.

@fanlv
Copy link
Collaborator

fanlv commented Jan 7, 2026

The license/CLA status check seems to have an issue. I’ll try closing and reopening the PR.

@fanlv fanlv closed this Jan 7, 2026
@fanlv fanlv reopened this Jan 7, 2026
@fanlv fanlv added this pull request to the merge queue Jan 7, 2026
Merged via the queue into coze-dev:main with commit 95d8ace Jan 7, 2026
9 checks passed
@codecov-commenter
Copy link

codecov-commenter commented Jan 7, 2026

Codecov Report

❌ Patch coverage is 0% with 2 lines in your changes missing coverage. Please review.

Files with missing lines Patch % Lines
backend/infra/document/ocr/impl/ppocr/ppocr.go 0.00% 1 Missing ⚠️
...d/infra/document/parser/impl/ppstructure/parser.go 0.00% 1 Missing ⚠️
Files with missing lines Coverage Δ
backend/infra/document/ocr/impl/ppocr/ppocr.go 0.00% <0.00%> (ø)
...d/infra/document/parser/impl/ppstructure/parser.go 0.00% <0.00%> (ø)
🚀 New features to boost your workflow:
  • ❄️ Test Analytics: Detect flaky tests, report on failures, and find test suite problems.
  • 📦 JS Bundle Analysis: Save yourself from yourself by tracking and limiting bundle sizes in JS merges.

@Bobholamovic Bobholamovic deleted the feat/add-header branch January 7, 2026 11:45
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