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

Update selenium image to 124.0 #42

Open
wants to merge 1 commit into
base: main
Choose a base branch
from

Conversation

koehnlein
Copy link
Sponsor

The Issue

The plugin uses a selenium docker image, which is about two years old and contains outdated Chromium version 101

How This PR Solves The Issue

This PR switches to a newer docker image with Chromium version 124. I also thought about using "latest" instead of a specific version. But I decided against it, because this would mean, the users always would use and pull new images with other Chromium versions which would result in unexpected behavior.

Manual Testing Instructions

Update the current plugin in your DDEV settings and execute ddev restart. This will trigger to pull the newer image.

Automated Testing Overview

Related Issue Link(s)

Release/Deployment Notes

The previously used image was about two years old and contains Chromium version 101.
@weitzman
Copy link
Collaborator

weitzman commented May 8, 2024

Hanging, just like it did at #31. i dont know the cause.

@koehnlein
Copy link
Sponsor Author

Thanks. I only search in open PRs for duplication

@andriokha
Copy link
Contributor

I think Selenium dropped compatibility with the JSON wire protocol a while back, but supporting the W3C dialect is still a WIP: https://www.drupal.org/project/drupal/issues/3432099.

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

Successfully merging this pull request may close these issues.

3 participants