Skip to content

feat(har): record securityDetails for API Requests - #32664

Merged
Simon Knott (Skn0tt) merged 1 commit into
microsoft:mainfrom
Skn0tt:har-api-request-security-details
Sep 18, 2024
Merged

feat(har): record securityDetails for API Requests#32664
Simon Knott (Skn0tt) merged 1 commit into
microsoft:mainfrom
Skn0tt:har-api-request-security-details

Conversation

@Skn0tt

Copy link
Copy Markdown
Member

While working on #32658 I discovered that we're recording securityDetails for browser requests, but not for API requests. This PR fixes that.

@github-actions

Copy link
Copy Markdown
Contributor

Test results for "tests 1"

2 failed
❌ [playwright-test] › playwright.artifacts.spec.ts:195:5 › should work with screenshot: only-on-failure & fullPage
❌ [playwright-test] › babel.spec.ts:135:5 › should not transform external

3 flaky ⚠️ [playwright-test] › ui-mode-test-setup.spec.ts:98:5 › should show errors in config
⚠️ [playwright-test] › ui-mode-test-progress.spec.ts:165:5 › should update tracing network live
⚠️ [webkit-library] › library/download.spec.ts:698:3 › should convert navigation to a resource with unsupported mime type into download

35491 passed, 659 skipped
✔️✔️✔️

Merge workflow run.

@Skn0tt
Simon Knott (Skn0tt) merged commit 8a97050 into microsoft:main Sep 18, 2024
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.

2 participants