Skip to content

Improve file blob upload diagnostics#32305

Merged
copyberry[bot] merged 1 commit into
mainfrom
copyberry/codex-internal-to-codex-oss/9ae2cb2a9aacec4df237b3ee79a6467a6b9107e7
Jul 11, 2026
Merged

Improve file blob upload diagnostics#32305
copyberry[bot] merged 1 commit into
mainfrom
copyberry/codex-internal-to-codex-oss/9ae2cb2a9aacec4df237b3ee79a6467a6b9107e7

Conversation

@copyberry

@copyberry copyberry Bot commented Jul 11, 2026

Copy link
Copy Markdown

Improve file blob upload diagnostics

Why

Blob upload failures previously surfaced the full signed upload URL and offered limited information for diagnosing transport and service errors.

What changed

  • Add a unique x-ms-client-request-id to each blob upload.
  • Report transport failures with the upload host, elapsed time, error category, and client request ID, while stripping the signed URL from the underlying error.
  • Report unsuccessful responses with Azure request and error IDs, and emit structured warning events with upload metadata and available Azure and Cloudflare identifiers.

Testing

  • Verify successful uploads include the client request ID header.
  • Verify response and transport errors expose diagnostic fields without leaking signed URL parameters or response bodies.

## Why

Blob upload failures previously surfaced the full signed upload URL and offered limited information for diagnosing transport and service errors.

## What changed

- Add a unique `x-ms-client-request-id` to each blob upload.
- Report transport failures with the upload host, elapsed time, error category, and client request ID, while stripping the signed URL from the underlying error.
- Report unsuccessful responses with Azure request and error IDs, and emit structured warning events with upload metadata and available Azure and Cloudflare identifiers.

## Testing

- Verify successful uploads include the client request ID header.
- Verify response and transport errors expose diagnostic fields without leaking signed URL parameters or response bodies.

GitOrigin-RevId: 9ae2cb2a9aacec4df237b3ee79a6467a6b9107e7
@copyberry copyberry Bot force-pushed the copyberry/codex-internal-to-codex-oss/9ae2cb2a9aacec4df237b3ee79a6467a6b9107e7 branch from 4a69ebe to 08ba14b Compare July 11, 2026 01:47
@copyberry copyberry Bot merged commit 08ba14b into main Jul 11, 2026
14 of 32 checks passed
@copyberry copyberry Bot deleted the copyberry/codex-internal-to-codex-oss/9ae2cb2a9aacec4df237b3ee79a6467a6b9107e7 branch July 11, 2026 01:48
@github-actions github-actions Bot locked and limited conversation to collaborators Jul 11, 2026
Sign up for free to subscribe to this conversation on GitHub. Already have an account? Sign in.

Labels

None yet

Projects

None yet

Development

Successfully merging this pull request may close these issues.

6 participants