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

Can not download chrome at Launch() #12

Closed
zjlx opened this issue Apr 24, 2020 · 5 comments
Closed

Can not download chrome at Launch() #12

zjlx opened this issue Apr 24, 2020 · 5 comments
Labels
bug When you are sure about it's a bug

Comments

@zjlx
Copy link

zjlx commented Apr 24, 2020

Describe the bug
Can not download chrome at Launch()

To Reproduce
Do not set launcher Bin , then it will download chrome.I am from china, the download url will forward taobao.org , but it is not download success.

Expected behavior
opening zip archive for reading: creating reader: zip: not a valid zip file[rod/lib/launcher] Download chromium from: https://npm.taobao.org/mirrors/chromium-browser-snapshots/Win/757680/chrome-win.zip [rod/lib/launcher]
I saw the dictionary https://npm.taobao.org/mirrors/chromium-browser-snapshots/Win/, does not have the version 757680,the latest is 737027.

Rod Version: v0.26.1

OS: Windows

@zjlx zjlx added the bug When you are sure about it's a bug label Apr 24, 2020
@ysmood ysmood added question Questions related to rod bug When you are sure about it's a bug and removed bug When you are sure about it's a bug labels Apr 24, 2020
@ysmood
Copy link
Member

ysmood commented Apr 24, 2020

That's a sad story 😂. Actually you can config the revision like this:

chrome := launcher.NewChrome()
chrome.Revision = "whatever you want"
bin, err := chrome.Get()
kit.E(err)

l := launcher.New().Bin(bin).Launch()

Or just set the Bin(your_local_chrome_bin) directly.

@ysmood
Copy link
Member

ysmood commented Apr 24, 2020

Seems like they only keep syncing the revision that puppeteer.js uses. Since it's an npm mirror.

Maybe we should find a better mirror.

BTW, if you installed the chrome on your local, it should work properly without downloading a new one. So you don't use the default location to install Chrome right?

@ysmood ysmood removed the question Questions related to rod label Apr 24, 2020
@ysmood
Copy link
Member

ysmood commented Apr 24, 2020

Anyway, I'll fix it and add a unit test for China. It shouldn't happen again.

ysmood added a commit that referenced this issue Apr 25, 2020
ysmood added a commit that referenced this issue Apr 25, 2020
@ysmood ysmood closed this as completed in 274a60c Apr 25, 2020
@ysmood
Copy link
Member

ysmood commented Apr 25, 2020

Published v0.26.2, should work fine now. Please check if you need to reopen it.

@zjlx
Copy link
Author

zjlx commented Apr 29, 2020

Published v0.26.2, should work fine now. Please check if you need to reopen it.

Now it works fine.

Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
bug When you are sure about it's a bug
Projects
None yet
Development

No branches or pull requests

2 participants