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

Incompatible with Selenoid #229

Open
grelner opened this issue Jun 27, 2024 · 4 comments
Open

Incompatible with Selenoid #229

grelner opened this issue Jun 27, 2024 · 4 comments

Comments

@grelner
Copy link
Contributor

grelner commented Jun 27, 2024

I've been trying to use thirtyfour with selenoid but the call to WebDriver::new fails.
The selenoid session api call is at /wd/hub/session while thirtyfour will always call /session . The reason is that thirtyfour uses Url::join to resolve url's with an argument that starts with '/'. Given webdriver url "http://localhost:4444/wd/hub" and "/session", Url::join will return http://localhost:4444/session
Personally I find the behavior of Url::join confusing, especially compared to Path::join, so this might not be intended behaviour?

@mallocdev
Copy link

@grelner Did you manage to use selenoid with thirtyfour?

@grelner
Copy link
Contributor Author

grelner commented Sep 2, 2024

@grelner Did you manage to use selenoid with thirtyfour?

Hi! Yes! The necessary patches are in my fork https://github.com/grelner/thirtyfour
This project was unmaintained back when I ran into this problem, that's why the patch just sits in my fork. But it seems there's a new maintainer now, so I'll make a PR when I'm back from vacation in a week so this gets moved into the main repo

@Vrtgs
Copy link
Owner

Vrtgs commented Sep 3, 2024

looking forward to it!

@Vrtgs
Copy link
Owner

Vrtgs commented Sep 19, 2024

any updates on this?

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

No branches or pull requests

3 participants