-
-
Notifications
You must be signed in to change notification settings - Fork 465
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
Softly download latest version on version mismatch #293
Comments
I also just found that that latest release according to this: https://chromedriver.storage.googleapis.com/LATEST_RELEASE |
That's already should work like you describe. If not - please provide the webdriver_manager version and example of code to reproduce. |
Oh, in that case I think it doesn't work like that when the version of Chrome browser is supplied manually. I have a different way of retrieving current Chrome browser version, which uses wind32api. I had to do it because I package my projects using pyinstaller and webdriver_manager uses subprocess to get chrome version, which is incompatible with pyinstaller and other packagers that create .exe out of python projects. I could see if I can change the way webdriver_manager looks for Chrome version on Windows to use win32api, if you guys are interested? This is a more reliable way that subprocess:
|
If you send 98.0.4758.81 to webdriver-manager as version, and chrome doesn't have a webdriver for it - so it fails - it's okay. |
I've tested both the release and the github version and it doesn't work like that. It tries to download this: and it fails. Looking at the code, I don't believe that's how it webdriver-manager works. You need to supply the full version string and it needs to match and existing chromedriver version. |
Provide code snippet how do you pass chromeversion to webdriver_manager. |
So I get the Chrome version using the above function and I then initialize webdriver_manager with a
|
In the meantime I've sorted it out by adding a function which finds the latest chromedriver version for a given major version number from It looks like this:
and I now call
but I think it would be interesting to see this type of graceful fallback directly in |
Even with your function - there is no way to get chromedriver version if chrome doesn't publish it. |
The problem that should be solved is the more powerfull windows version reader than webdriver_manager has now. win32api seems to be a huge dependency which I will not recommend to include to this library. But maintainer can have another opinion. We need just one function from this library. |
No, my code gets the latest published version for major browser version, so in this example it will get chromedriver '98.0.4758.80' for chrome browser '98.0.4858.81'. I already have it running in production. |
Today Google released Chrome 98.0.4758.81, but the latest Chromedriver version is 98.0.4758.48, which is compatible with this release. I propose webdriver_manager should download latest compatible version if there's a version mismatch, not just when the Chrome version is unknown.
The text was updated successfully, but these errors were encountered: