HTTP calls to HTTPS-only sites seems to stall instead of fail #7

Open
opened 2025-07-06 11:13:23 -07:00 by robert · 1 comment
Owner

Using the URL http://git.gelvin.dev causes the program to sit and wait. I'd expect it to either finish the call, or fail with some kind of protocol error. Marking as "need more info" because this will take a bit of time to investigate.

Using the URL `http://git.gelvin.dev` causes the program to sit and wait. I'd expect it to either finish the call, or fail with some kind of protocol error. Marking as "need more info" because this will take a bit of time to investigate.
robert added the Kind/Bug
Status
Need More Info
labels 2025-07-06 11:13:23 -07:00
Author
Owner

The program does time out... after > 2 minutes:

robert@computer:~/projects/gt-tools$ time gt-tool -u http://git.gelvin.dev -r robert/gt-tool list-releases
Error: WrappedReqwestErr(reqwest::Error { kind: Request, url: Url { scheme: "http", cannot_be_a_base: false, username: "", password: None, host: Some(Domain("git.gelvin.dev")), port: None, path: "/api/v1/repos/robert/gt-tool/releases/", query: None, fragment: None }, source: hyper::Error(Connect, ConnectError("tcp connect error", Os { code: 110, kind: TimedOut, message: "Connection timed out" })) })

real    2m9.693s
user    0m0.025s
sys     0m0.005s

The timeout should probably just be shortened so the user or automation doesn't waste so much time waiting for a non-existent endpoint.

The program *does* time out... after > 2 minutes: ```txt robert@computer:~/projects/gt-tools$ time gt-tool -u http://git.gelvin.dev -r robert/gt-tool list-releases Error: WrappedReqwestErr(reqwest::Error { kind: Request, url: Url { scheme: "http", cannot_be_a_base: false, username: "", password: None, host: Some(Domain("git.gelvin.dev")), port: None, path: "/api/v1/repos/robert/gt-tool/releases/", query: None, fragment: None }, source: hyper::Error(Connect, ConnectError("tcp connect error", Os { code: 110, kind: TimedOut, message: "Connection timed out" })) }) real 2m9.693s user 0m0.025s sys 0m0.005s ``` The timeout should probably just be shortened so the user or automation doesn't waste so much time waiting for a non-existent endpoint.
robert added the
Priority
Low
label 2025-07-06 15:13:21 -07:00
Sign in to join this conversation.