-
-
Notifications
You must be signed in to change notification settings - Fork 4
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
Adding WebDriver+Chrome server to the Docker Build #14
Comments
Thanks for opening your first issue here! Be sure to follow the relevant issue templates, or risk having this issue marked as invalid. |
This issue has been automatically marked as stale because it has not had recent activity. This might be due to missing feedback from OP. It will be closed if no further activity occurs. Thank you for your contributions. |
Agreed. Many of changedetection's features have the playwright browser as a dependency. Without it changedetection can only detect rudimentary changes, and many advanced features like browser steps, screenshots and logins are unavailable. |
Playwright won't provide muslc wheels and don't support source builds from pypi which makes life difficult. Could you try with In terms of Chrome we're not going to bundle Chromium in the container as it would basically double the size, and the webdriver depends on it so it's not like we can just install that part of it. If you want to add it yourself you can use https://github.com/linuxserver/docker-mods/tree/universal-package-install to do so based on the available packages here: https://pkgs.alpinelinux.org/packages?name=chromium-chromedriver&branch=v3.19 |
Actually, I was wrong, it nearly triples the size of the image
|
Disappointing that playwright doesn't build. That really clips the wings on changedetection. Thank you for the advice. I'll tinker with it when I find the time. |
The image I posted has playwright built from source from their git repo. It's a pain but not unworkable if it enables the functionality. |
This issue has been automatically marked as stale because it has not had recent activity. This might be due to missing feedback from OP. It will be closed if no further activity occurs. Thank you for your contributions. |
This issue is locked due to inactivity |
Is this a new feature request?
Wanted change
The Chrome/Javascript method requires a network connection to a running WebDriver+Chrome server, set by the ENV var 'WEBDRIVER_URL'.
Reason for change
Using the Screenshot functionality
Proposed code change
No response
The text was updated successfully, but these errors were encountered: