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

-notimeout has no effect after message : "Sending data..." #235

Closed
wighawag opened this issue Jul 23, 2015 · 4 comments
Closed

-notimeout has no effect after message : "Sending data..." #235

wighawag opened this issue Jul 23, 2015 · 4 comments

Comments

@wighawag
Copy link

without -notimeout:

haxelib submit lib.zip
Http connection timeout. Try running haxelib -notimeout <command> to disable timeout

with -notimeour

haxelib -notimeout submit lib.zip
Password :
Sending data....
Http connection timeout. Try running haxelib -notimeout <command> to disable timeout
@ibilon
Copy link
Member

ibilon commented Jul 23, 2015

The timeout parameter is ignored for the next step (processing) https://github.com/HaxeFoundation/haxelib/blob/master/src/tools/haxelib/Main.hx#L654 but normally not for this one.

Maybe the error message is too broad https://github.com/HaxeFoundation/haxelib/blob/master/src/tools/haxelib/Main.hx#L482 and the error is not timeout related? (just guessing here)

@juef17
Copy link

juef17 commented Dec 20, 2015

This seems to be an issue with install as well:

C:\Users\juef>haxelib install openfl
Downloading openfl-3,5,3.zip...
Download complete : 0 bytes in 10.5s (0KB/s)
Http connection timeout. Try running haxelib -notimeout <command> to disable timeout

C:\Users\juef>haxelib -notimeout install openfl
Downloading openfl-3,5,3.zip...
Download complete : 0 bytes in 10.5s (0KB/s)
Http connection timeout. Try running haxelib -notimeout <command> to disable timeout

@nadako
Copy link
Member

nadako commented Mar 10, 2016

Looks like -notimeout only has effect on remoting calls, not on general http request, for which it should be done using Http.cnxTimeout field.

@nadako nadako closed this as completed in 2b0cd09 Mar 10, 2016
@nadako
Copy link
Member

nadako commented Mar 10, 2016

This should work now.

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

No branches or pull requests

4 participants