feat: add request.body async getters #38179

New issue

Have a question about this project? Sign up for a free GitHub account to open an issue and contact its maintainers and the community.

Sign up for GitHub

By clicking “Sign up for GitHub”, you agree to our terms of service and privacy statement. We’ll occasionally send you account related emails.

Already on GitHub? Sign in to your account

Jump to bottom

Merged

yury-s merged 6 commits into microsoft:main from yury-s:request-body-async

Nov 11, 2025

Member

yury-s commented Nov 11, 2025 •

edited

Loading

Fixes #38150


          feat: add request.body async getters

9a5ec36

Reference microsoft#38150

This comment has been minimized.

Sign in to view

This comment has been minimized.

Sign in to view


          update wording in the docs

f340c9e

This comment has been minimized.

Sign in to view


          formatting

This comment has been minimized.

Sign in to view

This comment has been minimized.

Sign in to view

yury-s added 2 commits

November 11, 2025 12:54


          remove dup

35b6c3a


          Revert "remove dup"

4c4fad1

This reverts commit 35b6c3a.

This comment has been minimized.

Sign in to view


          mark sync methods as discouraged

7eaf707

dgozman approved these changes

View reviewed changes

This comment has been minimized.

Sign in to view

This comment has been minimized.

Sign in to view

This comment has been minimized.

Sign in to view

Contributor

github-actions bot commented Nov 11, 2025

Test results for "tests 1"

2 failed
❌ [playwright-test] › reporter-html.spec.ts:3005 › merged › execSync doesnt produce a second stdout attachment @macos-latest-node18-2
❌ [playwright-test] › runner.spec.ts:118 › should ignore subprocess creation error because of SIGINT @macos-latest-node18-2

4 flaky

⚠️ [firefox-library] › library/inspector/cli-codegen-1.spec.ts:1079 › cli codegen › should not throw csp directive violation errors `@firefox-ubuntu-22.04-node18`
⚠️ [firefox-page] › page/page-goto.spec.ts:83 › should work with Cross-Origin-Opener-Policy `@firefox-ubuntu-22.04-node18`
⚠️ [chromium] › headerView.spec.tsx:49 › should toggle filters `@web-components-html-reporter`
⚠️ [playwright-test] › ui-mode-test-output.spec.ts:80 › should show console messages for test `@windows-latest-node18-2`

40261 passed, 787 skipped

Merge workflow run.

yury-s merged commit 130a0fa into microsoft:main

59 of 62 checks passed

yury-s deleted the request-body-async branch

November 11, 2025 22:30

This was referenced Nov 11, 2025

[Ports]: Backport client side changes for 1.57 microsoft/playwright-python#2985

Closed

[Ports]: Backport client side changes for 1.57 microsoft/playwright-java#1848

Closed

[Ports]: Backport client side changes for 1.57 microsoft/playwright-dotnet#3242

Closed

Contributor

github-actions bot commented Nov 11, 2025

Test results for "MCP"

2 flaky

⚠️ [chrome] › mcp/cdp.spec.ts:24 › cdp server `@mcp-ubuntu-latest`
⚠️ [chrome] › mcp/cdp.spec.ts:35 › cdp server reuse tab `@mcp-ubuntu-latest`

2638 passed, 116 skipped

Merge workflow run.

dgozman added a commit to dgozman/playwright that referenced this pull request


          Revert "feat: add request.body async getters (microsoft#38179)"

80c42d6

This reverts commit 130a0fa.

dgozman added a commit that referenced this pull request


          Revert "feat: add request.body async getters (#38179)" (#38281)

46d5ea0

jchip pushed a commit to jchip/playwright that referenced this pull request


          feat: add request.body async getters (microsoft#38179)

2a9360e

jchip pushed a commit to jchip/playwright that referenced this pull request


          Revert "feat: add request.body async getters (microsoft#38179)" (micr…

…osoft#38281)

Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Labels

None yet