Skip to content
This repository has been archived by the owner on Apr 1, 2019. It is now read-only.

Updated chromedriver download methodology. #9

Merged
merged 2 commits into from Aug 26, 2014

Conversation

mars
Copy link
Contributor

@mars mars commented Oct 13, 2013

  • Utilize S3 bucket XML instead of parsing HTML
  • Encapsulate download URL logic in GoogleCodeParser
  • Always re-fetch zip archive, because filenames all look the same
  • Updated specs.

This fixes the issue I reported #7

There is another Pull Request open #8 but that simple patch will always returns chromedriver version 2.4 which will not alway be the newest.

* Utilize S3 bucket XML instead of parsing HTML
* Encapsulate download URL logic in GoogleCodeParser
* Always re-fetch zip archive, because filenames all look the same
* Updated specs.
@JanDintel
Copy link

👍 This fixed issue #7 for me. I tried pull request #8 as well, but that didn't work. Got the same error as in issue #7. This pull request fixed it, and I was able to run bundle exec chromedriver-update without problems.

@GeoffTidey
Copy link

works for me, thanks @mars 👍

flavorjones added a commit that referenced this pull request Aug 26, 2014
Updated chromedriver download methodology.
@flavorjones flavorjones merged commit 0ddedbc into flavorjones:master Aug 26, 2014
@flavorjones
Copy link
Owner

Merged and released in 0.0.6. Thank you, @mars!

Apologies to everyone for not seeing and responding to this 10 months ago.

Sign up for free to subscribe to this conversation on GitHub. Already have an account? Sign in.
Labels
None yet
Projects
None yet
Development

Successfully merging this pull request may close these issues.

None yet

4 participants