-
Notifications
You must be signed in to change notification settings - Fork 61
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
phantomjs alternative #124
Comments
There is now firefox headless https://mykzilla.org/2017/08/30/headless-firefox-in-node-js-with-selenium-webdriver/ or I guess more popularly, chrome headless. |
We'd support a move. I heard chrome headless is much faster. That said, we have no development time to devote to this at the moment unfortunately. And I don't feel a ton of urgency about it, we don't even keep up with phantom updates as it is.
… On Sep 21, 2017, at 6:52 PM, mark burdett ***@***.***> wrote:
There is now firefox headless https://mykzilla.org/2017/08/30/headless-firefox-in-node-js-with-selenium-webdriver/ or I guess more popularly, chrome headless.
—
You are receiving this because you are subscribed to this thread.
Reply to this email directly, view it on GitHub, or mute the thread.
|
Do we still need to support webkit/waitir? |
I think that's probably fine over here, yeah... |
has anyone started work on this? |
Not yet @j-ro. |
Thanks @wioux, us either, though it's starting to become more important for us. I'll let you know if it lands on my roadmap. Can you do the same, so we don't duplicate work? |
Definitely, I'll let you know. |
We're actually doing a bit of initial investigation work on this today, maybe tomorrow too. We'll let you know how it works. There may be just a drop-in replacement that works with capybera, if so, will be fairly easy.... |
Update here -- we have chromedriver running, but it's probably not quite ready for prime time. It works, but seeing some hard to debug timeout errors, and it's missing some features like blacklists. We're going to run it as an optional switch for certain yamls since it helps in some cases, but we're not going to entirely switch. If there's large appetitive for the code we can put together a PR, but it's very much a WIP. |
Hey @j-ro, this is becoming more important for us. Have you found a solution you like? |
No, we're still with phantom. Chromedriver works but not as consistently, and it doesn't have many hooks and options to debug and tune. We haven't looked at it since January, maybe that's changed, but we're not planning a switch. |
Ok, thanks for the insight. I'll see if anything's changed since then. |
Worth a shot -- it didn't really take us very long at all to drop in Chromedriver -- the hard part was getting it to work reliably. |
I'll chime in with my experience as I have worked with puppeteer, and phantomjs, and various selenium webdriver implementations like chromedriver and geckodriver. |
Just discovered @k-stewart 's work in #141 as well. |
Hi @efx. Our contact-congress work has moved over to EFForg/congress_forms_api to fix this and other issues. Sorry we didn't properly archive this repo -- I'm going to do that now. |
Thanks @wioux. I had found this repository from EFF's homepage, so we should probably update those link(s) as well. |
This repo is still not archived. We were about to roll out a system we have been working on for a while based on phantom of the capitol before noticing your comment :( |
Should we find an alternative to phantomjs? The maintainer has stepped down.
The text was updated successfully, but these errors were encountered: