Skip to content

Migrate from postData to postDataEntries in CdpHttpRequest#3126

Merged
kblok merged 3 commits intomasterfrom
implement-upstream-change-14640
Feb 12, 2026
Merged

Migrate from postData to postDataEntries in CdpHttpRequest#3126
kblok merged 3 commits intomasterfrom
implement-upstream-change-14640

Conversation

@kblok
Copy link
Member

@kblok kblok commented Feb 12, 2026

Summary

  • Ports upstream Puppeteer PR #14640
  • Uses postDataEntries (base64-encoded) from the CDP protocol instead of the deprecated postData field in Network.requestWillBeSent
  • Falls back to postData when postDataEntries is missing or empty, maintaining backward compatibility

Changes

  • PostDataEntry.cs — New messaging class for CDP PostDataEntry type
  • Request.cs (Messaging) — Added PostDataEntries property
  • CdpHttpRequest.cs — Added ReconstructPostData method that decodes base64 entries and merges them, replacing direct postData assignment
  • CdpHttpRequestTests.cs — Unit tests for reconstruction and fallback behavior

Test plan

  • New unit tests pass (reconstruct from entries + fallback)
  • Existing RequestPostData tests pass
  • Existing RequestContinue interception tests pass
  • Existing NetworkManager tests pass

🤖 Generated with Claude Code

kblok and others added 3 commits February 12, 2026 12:20
Use postDataEntries (base64-encoded) from CDP protocol instead of the
deprecated postData field. Falls back to postData when entries are missing.

Ports upstream puppeteer PR #14640.

Co-Authored-By: Claude Opus 4.6 <noreply@anthropic.com>
- Migrate implement-upstream-change and implement-bidi-feature from
  .claude/commands/ to .claude/skills/ format
- Add PuppeteerTest attributes to CdpHttpRequestTests to match upstream
- Remove obsolete .github/agents/ definitions

Co-Authored-By: Claude Opus 4.6 <noreply@anthropic.com>
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.

1 participant