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

"Failed to get stories" on FreeBSD #100

Open
phips opened this issue Dec 21, 2023 · 10 comments
Open

"Failed to get stories" on FreeBSD #100

phips opened this issue Dec 21, 2023 · 10 comments

Comments

@phips
Copy link

phips commented Dec 21, 2023

Hello,

Would you have any idea why the client fails to get stories on an ec2 FreeBSD machine, yet a curl works just fine please?

Installed via cargo install hackernews_tui --locked

Thanks!

Screenshot 2023-12-21 at 14 11 03 Screenshot 2023-12-21 at 14 11 29
@aome510
Copy link
Owner

aome510 commented Dec 22, 2023

Is this a consistent error which happens every time you start the application? Can you also share the application log in .cache folder?

@phips
Copy link
Author

phips commented Dec 22, 2023

Hello @aome510 — yes, everytime. Log should be attached, thanks!

hn-tui.log

@aome510
Copy link
Owner

aome510 commented Dec 24, 2023

Maybe related to algesten/ureq#637. I suspect this issue may involve the network setting of your machine. Can you checkout https://github.com/aome510/hackernews-TUI/tree/100-debug and run cargo run to see if there is a panic? The difference between that branch and latest main is

 fn main() {
+    let d = ureq::head("https://google.com").call().unwrap();
+    println!("{d:?}");
+

@phips
Copy link
Author

phips commented Dec 24, 2023

Screenshot 2023-12-24 at 15 23 03

A ha! That might be it, yes. There is ipv6 on this machine too. Hmm. Frustrating that the module doesn't work with the transport the world is trying to move towards, isn't it? 🤔

I'll try disabling it and see if it plays ball. But ultimately that's not the correct answer. Of course, I appreciate that is not this project's doing - ureq needs to get with the status quo 😄

@aome510
Copy link
Owner

aome510 commented Dec 24, 2023

Thanks for confirmation. I can switch to https://docs.rs/reqwest/latest/reqwest/ if ureq has that limitation

@phips
Copy link
Author

phips commented Dec 24, 2023

Is reqwest a bigger library than ureq? I don't suppose it ultimately matters. Shame ureq has such a fundamental thing as ipv6 missing really, isn't it.

@aome510
Copy link
Owner

aome510 commented Dec 24, 2023

Is reqwest a bigger library than ureq? I don't suppose it ultimately matters. Shame ureq has such a fundamental thing as ipv6 missing really, isn't it.

yeah, bigger and more dependencies. It's more popular and has more features as well. The main reason I chose ureq in the first place is that it's relatively minimal. Maybe, that's why it doesn't have a good support for ipv6.

@ryanmcgrath
Copy link

This may be resolved now since they deployed IPV6 support.

(The issue is primarily that ureq does not implement a happy-eyeballs type feature - i.e, it won't try both ipv4 and ipv6 at the same time and choose whichever one works.)

@aome510
Copy link
Owner

aome510 commented Jan 27, 2024

This may be resolved now since they deployed IPV6 support.

(The issue is primarily that ureq does not implement a happy-eyeballs type feature - i.e, it won't try both ipv4 and ipv6 at the same time and choose whichever one works.)

@phips can you try again with the new change? I haven't had a chance to switch to rewquest yet. If it works, maybe the switch is not needed.

@phips
Copy link
Author

phips commented Jan 28, 2024

Sadly nope @aome510 — still the same 😢

Screenshot 2024-01-28 at 12 50 48

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

3 participants