Skip to content

docs: clarify UndiciHeaders validation guidance#4832

Merged
mcollina merged 1 commit intomainfrom
docs/undiciheaders-validation-guidance
Feb 19, 2026
Merged

docs: clarify UndiciHeaders validation guidance#4832
mcollina merged 1 commit intomainfrom
docs/undiciheaders-validation-guidance

Conversation

@mcollina
Copy link
Member

@mcollina mcollina commented Feb 16, 2026

Summary\n- clarify that Undici performs protocol-level header syntax validation\n- document that callers must validate and sanitize user-provided header names and values\n- document that in the string[] header format, only indexed array elements are processed and extra array object properties are ignored\n\n## Testing\n- npm run lint

@codecov-commenter
Copy link

codecov-commenter commented Feb 16, 2026

Codecov Report

✅ All modified and coverable lines are covered by tests.
✅ Project coverage is 93.20%. Comparing base (0a23610) to head (5957fd1).

Additional details and impacted files
@@ Coverage Diff @@ ## main #4832 +/- ## ========================================== + Coverage 93.19% 93.20% +0.01%  ========================================== Files 109 109 Lines 34222 34222 ========================================== + Hits 31893 31897 +4  + Misses 2329 2325 -4 

☔ View full report in Codecov by Sentry.
📢 Have feedback on the report? Share it here.

🚀 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.
Copy link
Contributor

@Uzlopak Uzlopak left a comment

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

This is important, but I fear people will not notice it. I think we should put it into a warning box, if possible

@mcollina mcollina merged commit 5bdd322 into main Feb 19, 2026
34 of 37 checks passed
@mcollina mcollina deleted the docs/undiciheaders-validation-guidance branch February 19, 2026 20:12
@github-actions github-actions bot mentioned this pull request Mar 12, 2026
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Labels

None yet

5 participants