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

Support reconnecting to an existing session #100

Draft
wants to merge 2 commits into
base: main
Choose a base branch
from

Conversation

travisbrown
Copy link
Contributor

This is a sketch of a minimal fix for #57 that just copies what with_capabilities is doing but without issuing the NewSession command.

I've confirmed that this implementation does what I need right now (allows me to have a CLI that doesn't have to log in to a site on every invocation, for example), but it should probably also do things like try to read the value of is_legacy off the WebDriver status for the session. It also needs tests, etc.

@jonhoo
Copy link
Owner

jonhoo commented Aug 6, 2020

Nice! I wonder if we could easily share some of this code between new and with_session. In theory I believe that should be doable.

@JosiahBull
Copy link

Is this still being worked on?

@jonhoo jonhoo mentioned this pull request Jan 6, 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.

None yet

3 participants