Skip to content
This repository was archived by the owner on Sep 11, 2025. It is now read-only.

Conversation

BitFis
Copy link

@BitFis BitFis commented May 30, 2021

Hey,

I Run into the issue, that the chrome devtool connection (connectOverCDP) as defined in https://playwright.dev/docs/api/class-browsertype/#browsertypeconnectovercdpparams is not supported. This addition would allow such a support by checking for the endpointURL argument in options and selecting connect function accordingly.

I hope this helps improve the extension.

Great work ❤️

Select connect function based on endpointURL being set
@mmarkelov
Copy link
Member

@BitFis could you please add endpointURL to ConnectOptions?

@coveralls
Copy link

coveralls commented Jun 2, 2021

Pull Request Test Coverage Report for Build 900367984

  • 0 of 0 changed or added relevant lines in 0 files are covered.
  • No unchanged relevant lines lost coverage.
  • Overall coverage remained the same at 94.798%

Totals Coverage Status
Change from base Build 898496421: 0.0%
Covered Lines: 106
Relevant Lines: 109

💛 - Coveralls

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.

3 participants