Skip to content

Conversation

@sadym-chromium
Copy link
Collaborator

@sadym-chromium sadym-chromium commented Aug 22, 2025

Blocked on w3c/webdriver#1920

Spec: w3c/webdriver-bidi#988

Breaking change: proxy parameter does not allow for schema.

@sadym-chromium sadym-chromium added puppeteer Run Puppeteer test when added to PR update-expectations Update WPT expectations labels Aug 22, 2025
@OrKoN
Copy link
Collaborator

OrKoN commented Aug 22, 2025

It looks like it would not be compatible with Puppeteer?

browser-automation-bot added a commit to browser-automation-bot/chromium-bidi that referenced this pull request Aug 22, 2025
@sadym-chromium sadym-chromium changed the title feat: validate browser.createUserContext:proxy feat!: validate browser.createUserContext:proxy Aug 22, 2025
@sadym-chromium
Copy link
Collaborator Author

It looks like it would not be compatible with Puppeteer?

It is compatible but it's a breaking change. In Puppeteer, we will have to parse the proxy string and strip out the schema from proxies per schema.

@sadym-chromium
Copy link
Collaborator Author

It looks like it would not be compatible with Puppeteer?

yeah, it's a breaking change due to w3c/webdriver#1920

@sadym-chromium sadym-chromium added the DO NOT MERGE Don't merge the marked PR label Aug 22, 2025
@sadym-chromium sadym-chromium marked this pull request as draft August 25, 2025 08:15
@sadym-chromium sadym-chromium force-pushed the sadym/browser.createUserContext.proxy branch from ff83583 to aa92ba3 Compare August 29, 2025 11:48
@sadym-chromium sadym-chromium force-pushed the sadym/browser.createUserContext.proxy branch from aa92ba3 to c6b2f1d Compare September 12, 2025 11:03
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Labels

DO NOT MERGE Don't merge the marked PR puppeteer Run Puppeteer test when added to PR update-expectations Update WPT expectations

Projects

None yet

Development

Successfully merging this pull request may close these issues.

3 participants