Preserve header case in IOClient#1918
Conversation
Closes #609 Some non-compliant servers misbehave with the default handling of headers in the `dart:io` HTTP implementation. Well behaved servers should not be impacted, but other use cases which require a particular formatting can be handled by always preserving header case.
PR HealthLicense Headers ✔️
All source files should start with a license header. Unrelated files missing license headers
This check can be disabled by tagging the PR with Breaking changes ✔️
This check can be disabled by tagging the PR with Coverage ✔️
This check for test coverage is informational (issues shown here will not fail the PR). This check can be disabled by tagging the PR with API leaks ✔️The following packages contain symbols visible in the public API, but not exported by the library. Export these symbols or remove them from your publicly visible API.
This check can be disabled by tagging the PR with
Unused Dependencies
|
| Package | Status |
|---|---|
| http | ❗ Show IssuesThese packages may be unused, or you may be using assets from these packages: |
For details on how to fix these, see dependency_validator.
This check can be disabled by tagging the PR with skip-unused-dependencies-check.
Changelog Entry ✔️
| Package | Changed Files |
|---|
Changes to files need to be accounted for in their respective changelogs.
This check can be disabled by tagging the PR with skip-changelog-check.
brianquinlan
left a comment
There was a problem hiding this comment.
Is it worth adding a test for this?
Can't hurt. Done. |
Closes #609
Some non-compliant servers misbehave with the default handling of
headers in the
dart:ioHTTP implementation. Well behaved serversshould not be impacted, but other use cases which require a particular
formatting can be handled by always preserving header case.