Skip to content
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.

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

fead: add Browser.disconnect method #94

Merged
merged 1 commit into from
Oct 4, 2024
Merged

Conversation

lino-levan
Copy link
Owner

I was surprised to see that playwright doesn't have this (microsoft/playwright#4956). Looks like we'll have to defer to puppeteer's judgement here.

@lino-levan lino-levan linked an issue Oct 4, 2024 that may be closed by this pull request
@lino-levan lino-levan merged commit 08d9573 into main Oct 4, 2024
3 checks passed
@lino-levan lino-levan deleted the feat-browser-disconnect branch October 4, 2024 14:53
@chaosharmonic
Copy link
Contributor

chaosharmonic commented Oct 6, 2024

So it looks like Playwright does have a browser.connect() method, which could address the other part of the attached issue (resource disposal closing the browser instead of disconnecting)

It might also explain #86

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.

Add disconnect handling for use with manually set WebSocket endpoints
2 participants