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

[🚀 Feature]: Updating firefox for the standalone selenium 2 image #2454

Closed
uuf6429 opened this issue Nov 1, 2024 · 12 comments
Closed

[🚀 Feature]: Updating firefox for the standalone selenium 2 image #2454

uuf6429 opened this issue Nov 1, 2024 · 12 comments

Comments

@uuf6429
Copy link

uuf6429 commented Nov 1, 2024

Feature and motivation

At the Mink project, we are still supporting Selenium 2 and in some recent tests we added retroactively, we found that they fail for the Firefox / Selenium 2 image due to Firefox 47 (which is 8 years old - latest is 132, by comparison).

Is there any chance of pushing and tagging an updated image (not necessarily with the latest Firefox, just a small update from 47)?

Alternatively, we can somehow do it ourselves, but I've figured this could be in the interest of the community.

Usage example

This would help supporting users of Selenium 2.

Copy link

github-actions bot commented Nov 1, 2024

@uuf6429, thank you for creating this issue. We will troubleshoot it as soon as we can.


Info for maintainers

Triage this issue by using labels.

If information is missing, add a helpful comment and then I-issue-template label.

If the issue is a question, add the I-question label.

If the issue is valid but there is no time to troubleshoot it, consider adding the help wanted label.

If the issue requires changes or fixes from an external project (e.g., ChromeDriver, GeckoDriver, MSEdgeDriver, W3C), add the applicable G-* label, and it will provide the correct link and auto-close the issue.

After troubleshooting the issue, please add the R-awaiting answer label.

Thank you!

@VietND96
Copy link
Member

VietND96 commented Nov 3, 2024

@aik099
Copy link

aik099 commented Nov 3, 2024

@VietND96 we use this command to start Docker:

docker run --net host --name selenium --volume /dev/shm:/dev/shm --volume ./vendor/mink/driver-testsuite/web-fixtures:/fixtures --shm-size 2g selenium/standalone-firefox:2.53.1 &> ./logs/selenium.log &

If I'm not mistaken the selenium/standalone-firefox:2.53.1 is the tag name.

@VietND96
Copy link
Member

VietND96 commented Nov 3, 2024

I see, but when upgrading to the new Firefox, we also need to consider the Geckodriver version. Moreover, if you are still using Selenium 2, how do we ensure the combination of those 3 works together?

@uuf6429
Copy link
Author

uuf6429 commented Nov 3, 2024

@VietND96 before reporting this issue I noticed that there is a script that somehow builds an image based on minimal input (selenium version, browser type, browser version):
https://github.com/SeleniumHQ/docker-selenium?tab=readme-ov-file#build-the-images-with-specific-versions

I'm not deeply familiar with this project, so I cannot answer how images are being tested.

However: doing a minimal upgrade should probably not break too many things and secondly our test suite would be able to say if it worked fine or not.

@aik099
Copy link

aik099 commented Nov 3, 2024

@VietND96 I can tell, that Firefox 132.0 + geckodriver 0.35.0 combination works with Selenium 2.

Both versions mentioned are current.

@VietND96
Copy link
Member

VietND96 commented Nov 3, 2024

Yes, that note is written for images built on top of Selenium 4. For older version, I am not sure it will work, since there is a big gaps in env vars configuration between v2 and v4.
Anyhow, which version of Firefox do you want to upgrade in v2 image now?

@VietND96
Copy link
Member

VietND96 commented Nov 3, 2024

@VietND96 I can tell, that Firefox 132.0 + geckodriver 0.35.0 combination works with Selenium 2.

Ok, let me try to see the patch can be done

@uuf6429
Copy link
Author

uuf6429 commented Nov 3, 2024

Yes, that note is written for images built on top of Selenium 4. For older version, I am not sure it will work, since there is a big gaps in env vars configuration between v2 and v4.

Sorry to hear that, I hoped it would be straightforward.

Anyhow, which version of Firefox do you want to upgrade in v2 image now?

At least Firefox 92, but higher (as @aik099 reported) wouldn't hurt. :)

Ok, let me try to see the patch can be done

Thanks for your time!

@diemol
Copy link
Member

diemol commented Nov 3, 2024

You need to check out the tag for that version https://github.com/SeleniumHQ/docker-selenium/tree/2.53.1, and then modify the Firefox version there https://github.com/SeleniumHQ/docker-selenium/blob/2.53.1/NodeFirefox/Dockerfile.txt#L8. The GeckoDriver version is below that. The README shows how to do the build process https://github.com/SeleniumHQ/docker-selenium/blob/2.53.1/README.md

I am unsure if everything will work because those images rely on old packages.

Please note that Mozilla does not support Selenium 2: https://firefox-source-docs.mozilla.org/testing/geckodriver/Support.html

I will close this because we are focused on the Selenium 4 images and the Helm chart. If you need help, you can open a discussion. Thanks!

@diemol diemol closed this as not planned Won't fix, can't repro, duplicate, stale Nov 3, 2024
@uuf6429
Copy link
Author

uuf6429 commented Nov 3, 2024

That's slightly disappointing, but understandable. 😅

Thanks for the pointers though.

Copy link

github-actions bot commented Dec 4, 2024

This issue has been automatically locked since there has not been any recent activity after it was closed. Please open a new issue for related bugs.

@github-actions github-actions bot locked and limited conversation to collaborators Dec 4, 2024
Sign up for free to subscribe to this conversation on GitHub. Already have an account? Sign in.
Projects
None yet
Development

No branches or pull requests

4 participants