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

download.default_directory no longer working #44

Closed
muppet3000 opened this issue Feb 13, 2023 · 8 comments
Closed

download.default_directory no longer working #44

muppet3000 opened this issue Feb 13, 2023 · 8 comments

Comments

@muppet3000
Copy link

Since the upgrade to Chromedriver 110.0.5481.77 the download.default_directory setting is no longer honoured.

This is a bug upstream: SeleniumHQ/selenium#11637

It's up to you whether you'd consider reverting back to the previous release or waiting for the fix, however since we use the download directory extensively in our work we've had to revert back to a previous version of the docker image that we had stored locally.

Perhaps it would be nice to consider an additional naming convention for the docker images to allow use of specific Chromedriver releases e.g.
jdk-11 and also jdk-11-<chromedriver_version>
Therefore in the event of such an issue we can pin to specific previous versions until an update is available?

@markhobson
Copy link
Owner

Hi, sorry to hear that. There has been previous discussions about tagging the image for browser versions (see #2), but I'm not sure that I have the bandwidth to maintain tags beyond the JDK versions.

As a workaround, have you considered using digests to fix the image in your builds? For example:

$ docker run -it --rm markhobson/maven-chrome:jdk-11@sha256:d5851295c6815c0d21d4c981f4431bcb47ffe53048ebc0f2a4deec7cf7ecf78e chromedriver --version
ChromeDriver 110.0.5481.77 (65ed616c6e8ee3fe0ad64fe83796c020644d42af-refs/branch-heads/5481@{#839})

$ docker run -it --rm markhobson/maven-chrome:jdk-11@sha256:4b34e00416695937de6190967ae6c92dfcdb80519d29801e5b557d43395f3991 chromedriver --version
ChromeDriver 109.0.5414.74 (e7c5703604daa9cc128ccf5a5d3e993513758913-refs/branch-heads/5414@{#1172})

Hopefully this can tide you over until ChromeDriver is fixed.

@muppet3000
Copy link
Author

I'd forgotten actually that you could do that. We'll use that in the meantime. In terms of bandwidths, I completely understand, I'll see whether I have some time and could try contributing a PR that would extend your ci pipelines to do this automatically for you.
Thanks for taking the time to respond at least.
Can I ask that this ticket remains open until the next chromedriver bump happens so I can track when a resolution is available?

@markhobson
Copy link
Owner

It is an often forgotten feature, handy at times though!

Thanks, I'd be interested in automatically tagging images with the browser version. Although note that Chrome and ChromeDriver versions don't always match exactly, e.g. right now the latest versions are Chrome 110.0.5481.100 and ChromeDriver 110.0.5481.77.

Sure, no problem to keep this open until your issue is resolved. Have you got a link to the underlying ChromeDriver bug?

@muppet3000
Copy link
Author

Tagged in my original comment.

@markhobson
Copy link
Owner

That seems to be the Selenium issue, where they said to raise one in ChromeDriver instead.

@muppet3000
Copy link
Author

So it is, my bad, I'll ask in that other thread to see if anyone actually raised the issue.

@markhobson
Copy link
Owner

Hi @muppet3000, has this issue been resolved now? I'll close this if so.

@markhobson
Copy link
Owner

Closing due to inactivity.

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

2 participants