Use CURLOPT_CONNECTTIMEOUT if timeout too big

[LorenzoLeonardo]

If the time that was converted into millisecond is more than the capacity of c_long, it will overflow, thus we need to convert it into seconds and use CURLOPT_CONNECTTIMEOUT.

[LorenzoLeonardo]

I think tests may be failing though? (perhaps because curl is too old in the container?)

I will try to use i32:MAX instead of std::i32:MAX https://doc.rust-lang.org/std/i32/constant.MAX.html

[LorenzoLeonardo]

oh, that's weird, if I run the Unit Test locally this will succeed.

[alexcrichton]

I believe the failure is because c_ulong is u64 on Linux so u64::MAX fits. I think you'll want to specify a larger amount than u64::MAX milliseconds to guarantee failure.

[LorenzoLeonardo]

will try to use from_secs() to guarantee max value

[LorenzoLeonardo]

It seems there are test error not related to the change.

[LorenzoLeonardo]

it is ok now