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

Default timeout is too short. #1333

Closed
nwin opened this issue Feb 22, 2015 · 1 comment · Fixed by #1341
Closed

Default timeout is too short. #1333

nwin opened this issue Feb 22, 2015 · 1 comment · Fixed by #1341

Comments

@nwin
Copy link

nwin commented Feb 22, 2015

The default timeout if 30 s. The upload limit of crates.io is 10 MB. Assuming an internet connection with about 1mbit/s, it would roughly take 60 s to download a package. That does not fit.

Actually I do not even see how much sense a timeout makes, if the download works fine. Cargo should probably only use a timeout for establishing the connection but not for the download itself.

In principle I would even say this should fixed in rust-curl. The default timeout should be set to 0.

@alexcrichton
Copy link
Member

I agree that the limit should probably be raised.

alexcrichton added a commit to alexcrichton/cargo that referenced this issue Feb 24, 2015
bors added a commit that referenced this issue Feb 24, 2015
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 a pull request may close this issue.

2 participants