Alan-Jowett / sonde / 26608068360

82%

Build Type

push

github

Committed by web-flow

Commit Message

azure-handler: inject synthetic Server header for Azure Table Storage stamps that omit it (#1090)

Some Azure Table Storage stamps do not return a Server HTTP response
header, but azure_storage 0.21.0 CommonStorageResponseHeaders
unconditionally parses it via server_from_headers(headers)?, failing
with "header not found server" (azure-sdk-for-rust#4489).

This was verified against two storage accounts in the same region/SKU:
- sts2jb46h7tqwje (newer stamp) - never returns Server header
- st7ikf25ddv4vmg (older stamp) - always returns Server header

Fix: inject an InjectServerHeaderPolicy into the
TableServiceClientBuilder pipeline (via per_call_policies) that
adds a synthetic Server: Windows-Azure-Table/1.0 Microsoft-HTTPAPI/2.0
header to responses when one is missing. This uses the SDK's intended
policy extension mechanism and is trivially removable when the upstream
fix lands.

Closes #1089

Signed-off-by: Alan Jowett <alan.jowett@microsoft.com>
Co-authored-by: Alan Jowett <alan.jowett@microsoft.com>
Co-authored-by: Copilot <223556219+Copilot@users.noreply.github.com>

Coverage Stats

42 of 48 new or added lines in 1 file covered. (87.5%)

1 existing line in 1 file now uncovered.

37919 of 46142 relevant lines covered (82.18%)

251.18 hits per line