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

0.9.6 "connect" error on Android #427

Closed
thomcc opened this issue Jan 10, 2019 · 5 comments

Comments

Projects
None yet
3 participants
@thomcc
Copy link

commented Jan 10, 2019

In the most recent patch build we're seeing a Connection error that describes itself as "io error" (and barely any other information). We've only seen it on Android. Pinning the version to 0.9.5 fixes the issue.

Unfortunately, due to difficulty getting symbolicated stacks off of android, we don't have anything more than to say that the specific error we're getting formats as follows:

Error { kind: Hyper(Error { kind: Connect, cause: Custom { kind: Other, error: StringError("io error") } }), url: Some("https://latest.dev.lcip.org/.well-known/fxa-client-configuration") }

If you feel like debugging this yourself, I can give you steps to get set up, but it's a little involved if you want to do it in such a way that you have any insight into the rust code running under the hood. I'm also willing to help debug this in other ways, as we would rather not stay pinned to an old version forever.

@seanmonstar

This comment has been minimized.

Copy link
Owner

commented Jan 10, 2019

I believe this error is coming from trust-dns, since error claims it happened during Connect, and 0.9.6 replaced the blocking resolver with an async one. Additionally, I see in trust-dns-resolver that io::Errors toss their details and just say "io error".

Apart from that, I don't know why exactly it is failing. I'm thinking it may be loading the system conf that is failing...

@thomcc

This comment has been minimized.

Copy link
Author

commented Jan 10, 2019

Ah, so that's where it comes from. I can try getting more info out of the io error tomorrow if it would help.

@seanmonstar seanmonstar changed the title 0.9.6 connection error on Android 0.9.6 "connect" error on Android Jan 10, 2019

@seanmonstar seanmonstar added the bug label Jan 10, 2019

@seanmonstar

This comment has been minimized.

Copy link
Owner

commented Jan 10, 2019

In #429, we should be able to see better error details. If the error is around loading the system conf (/etc/resolv.conf), then maybe we can get that fixed in trust-dns, or at least in reqwest we can add a config flag when on Android to use the built-in getaddrinfo resolver instead.

@thomcc

This comment has been minimized.

Copy link
Author

commented Jan 10, 2019

I get Error { kind: DnsSystemConf(Os { code: 2, kind: NotFound, message: "No such file or directory" }), url: None } with this. So it looks like you were right, it is failing to load the system conf.

@kud1ing

This comment has been minimized.

Copy link

commented Jan 11, 2019

Hm, on a Mac and in the Playground i see similar effects: #435

Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
You can’t perform that action at this time.