Join GitHub today
GitHub is home to over 31 million developers working together to host and review code, manage projects, and build software together.
Sign upAll network requests on Windows 7 fail on 0.1.9. #423
Comments
This comment has been minimized.
This comment has been minimized.
liigo
commented
May 9, 2016
|
on Windows 10, I got this error (and i think this is also a network error, based on the os error code 10054):
|
This comment has been minimized.
This comment has been minimized.
|
@liigo That unicode inner message says "远程主机强迫关闭了一个现有的连接。" / "It was forcibly closed by the remote host an existing connection.". I'm guessing the PoisonError you are seeing is from the Mutex around TlsStream, but I can't imagine how. @kevinpoitra's error looks like
The error information surfaced in these error messages isn't great. I want to be able to see type names for every error. And ProtocolError isn't providing a lot of information. cc @alexcrichton error reporting stuff. @sfackler Do you understand what's happening with the errors on windows? |
This comment has been minimized.
This comment has been minimized.
|
I do not have a great understanding of the schannel stuff unfortunately. |
sfackler
closed this
May 9, 2016
sfackler
reopened this
May 9, 2016
This comment has been minimized.
This comment has been minimized.
|
Having done a marginal amount of poking, I was able to yield this (some of the debug messages mine):
I don't actually know anything about schannel, but maybe this info can save some time? |
This comment has been minimized.
This comment has been minimized.
|
I've reproduced the panic @lligo sees, calling |
This comment has been minimized.
This comment has been minimized.
|
And I got a better stack trace
|
This comment has been minimized.
This comment has been minimized.
|
I'm running into lots of weird problems. Downloads randomly hang for a very long time. |
This comment has been minimized.
This comment has been minimized.
|
I've also experienced long delays before the download starts, although I haven't had it fail as described here. |
This comment has been minimized.
This comment has been minimized.
pravic
commented
May 9, 2016
|
Is 0.1.8 works well? How to downgrade to it? Or it is website issue? |
This comment has been minimized.
This comment has been minimized.
|
@pravic 0.1.8 works but is completely insecure. There are also no release archives to download it from - the builds are gone. |
This comment has been minimized.
This comment has been minimized.
|
@brson for now it might make sense to back out of native-tls back to openssl, but include the verify stuff. I'm in the process of rewriting the schannel library and that may resolve these weird issues as well. |
This comment has been minimized.
This comment has been minimized.
|
That may be best, yeah. |
This comment has been minimized.
This comment has been minimized.
|
The stack trace in my previous message looks fairly nonsensical. Lots of interleaved frames that look unrelated. |
This comment has been minimized.
This comment has been minimized.
|
I've made some progress debugging the schannel integration and have identified several bugs. After fixing it seems to be performing better locally, but I won't have a patch together until tomorrow. |
brson
referenced this issue
May 10, 2016
Merged
Don't panic when the NativeSslStream lock is poisoned #429
This comment has been minimized.
This comment has been minimized.
|
I've made the fix for the long delays and eliminated the panic from mutex poisoning, but I doubt I've found the protocolerror from the OP. Schannel is a typically complicated Win32 API and it's really hard to figure out how to use it correctly. It looks to me like schannel-rs is not doing the shutdown sequence, and I'm sure it's got more bugs. |
This comment has been minimized.
This comment has been minimized.
pravic
commented
May 10, 2016
•
Guess its better to use the unsecure version than broken one. Or at least an option to use http for non-paranoics. Offtop :( |
This comment has been minimized.
This comment has been minimized.
|
@pravic Yeah, plain HTTP would be a good workaround. Too bad I disabled it in the same PR that I enabled the broken TLS! I'll make another PR to enable plain HTTP. |
This comment has been minimized.
This comment has been minimized.
toothbrush7777777
commented
May 10, 2016
•
|
I've got the same error on Windows 7 Ultimate SP1 (Core i3 64-bit, 8 GiB RAM), using
|
This comment has been minimized.
This comment has been minimized.
|
For the record I just got the latest x86_64-pc-windows-msvc rustup-init on Windows 10 x64 and had no issues with it downloading and installing stuff. |
This comment has been minimized.
This comment has been minimized.
Arnavion
commented
May 11, 2016
|
@brson Unfortunately 0.1.10 doesn't fix the issue completely. Even though setting the two RUSTUP env vars to With a build where I added some logging to dump the env var in
|
This comment has been minimized.
This comment has been minimized.
Arnavion
commented
May 11, 2016
|
@pravic I think that error means you're still using 0.1.9. |
This comment has been minimized.
This comment has been minimized.
pravic
commented
May 11, 2016
|
Yep. |
This comment has been minimized.
This comment has been minimized.
Arnavion
commented
May 11, 2016
|
And @brson released 0.1.10 to fix that problem, which as I said in my comment doesn't fix the problem. |
This comment has been minimized.
This comment has been minimized.
Indeed :( |
This comment has been minimized.
This comment has been minimized.
Arnavion
commented
May 11, 2016
|
(I worked around it by just doing |
This comment has been minimized.
This comment has been minimized.
pravic
commented
May 13, 2016
|
Is it fixed? 0.1.11 and today's 0.1.12 seems to work well. |
This comment has been minimized.
This comment has been minimized.
toothbrush7777777
commented
May 17, 2016
|
@pravic The current version works fine for me. |
This comment has been minimized.
This comment has been minimized.
|
Tried out 0.1.12 on the machine that was originally having issues, all seems to be working fine now. |
kevinpoitra commentedMay 8, 2016
I was using 0.1.8 up until yesterday. I ran rustup self update, it updated to 0.1.9, and now anything that makes a network request fails. Specifically, rustup throws an error here. I then tried uninstalling rustup via rustup self uninstall, and then downloaded rustup-init.exe and ran that, which also is giving the same error as it attempts to download the stable channel's current hash. Here's the error itself.
I want to say this is due to this rust-native-tls issue, but I'm not certain.
For reference, I'm on Windows 7.