-
Notifications
You must be signed in to change notification settings - Fork 81
Use a compatible version of chromedriver #78
Comments
@cbandy Thanks for raising this suggestion. Can you help me understand why pinning to a compatible version on your dev system wouldn't be a sufficient solution for you? Just trying to understand the full context of the challenge you're seeing. This seems like a reasonable use case to cover, but since this could be addressed today by pinning to a specific version on your system, I would like to take a moment to explore the complexity of the suggested solution, and ask for help from someone who is feeling this pain today. The suggested solution would need to take into account a few different configuration dimensions:
My experience has been that implementing something like this ends up being a game of proverbial whack-a-mole, with new unexpected configurations of development systems being raised as bugs, and so I'm extremely hesitant to go in this direction. That said, if someone is willing to work with me on this approach, I'd be willing to consider a PR along these lines. Please let me know if you're willing to put in some time with me. |
It does resolve the errors. My feature request here is that chromedriver-helper not introduce the errors.
I assumed that because I did not specify these paths or choices in my configuration something was already doing this detection already. I see now that it happens within chromedriver itself.
I'm afraid my expertise is not in the relationship between Selenium, chrom{ium,e}, and chromedriver. For myself, I think I would be satisfied with some guidance in this project's documentation.
While looking at chromedriver, I noticed that the executable only recently provides an indication of which version it supports. |
@cbandy 👍 I will try to clarify this behavior in the README. |
The chromedriver documentation seems to have recently changed, it looks like the major version of chromedriver must match the version of Chrome now - since Chrome/chromedriver 73 there is no mention of an overlap where different versions of Chrome are supported simultaneously. I have an environment running Chrome 72 that has downloaded chromedriver 73 via chromedriver-helper and it mostly works but a couple of tests appear to crash Chrome; I will try upgrading Chrome to see if that solves the issue. http://chromedriver.chromium.org/downloads/version-selection has an algorithm that perhaps chromedriver-helper should follow to download the correct version of chromedriver? |
@longwave I think you're asking for a broader change which requires introspection on your system to determine what versions of chromedriver are compatible ... can you please open a new issue about that? |
@flavorjones I disagree... this gem appears to be "blindly" installing the latest version of chrome driver (Which is 74 right now) along with the latest stable version of Chrome (currently 73, 74 not due to stable for 41 days). There is another issue (#79 ) which touches on a similar point of version matching. |
Hey all, as I mentioned on Feb 7, I'm open to a PR for this but I have concerns about the likely incompleteness of the proposed solutions. Happy to work with someone, but I honestly don't think I'll be able to get to this for at least a few days. |
I'm going to close this issue, as the gem is being deprecated in favor of webdrivers, see #83. |
Similar to #77, the latest release of chromedriver is incompatible with the version of Chromium on my system.
It would be great if chromedriver-helper checked the version of Chromium and chose a working version automatically.
Specifically, I have
And ran into errors like
The text was updated successfully, but these errors were encountered: