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

Improve IP-based geolocation #1073

Open
tr1xem opened this issue Mar 23, 2025 · 4 comments
Open

Improve IP-based geolocation #1073

tr1xem opened this issue Mar 23, 2025 · 4 comments

Comments

@tr1xem
Copy link

tr1xem commented Mar 23, 2025

I guess it should return weather data according to the location it got from ip??

Its returing

Weather report: not found

But While passed with location (that too fetched from ip) it returns correct weather

@chubin
Copy link
Owner

chubin commented Mar 23, 2025

It is possible that it works when you specify your location manually. It just means that the service can't detect your location based on your IP address.

Can you please verify that it is exactly this problem?

$ curl ifcfg.me
# to get your IP as it is visible from outside

$ curl wttr.in/@<YOUR_IP>

If it returns not-found again, the problem is exactly what I meant. If it resolves your location, though, it means there should be a bug.

Please check.

@tr1xem
Copy link
Author

tr1xem commented Mar 23, 2025

🞈 curl wttr.in/@{myip}
Weather report: not found

This is the output

@chubin
Copy link
Owner

chubin commented Mar 23, 2025

Okay, then everything is as expected. Anyway, thank you for reporting, and I hope that we will increase the quality of our IP-based geolocation in the future. Stay tuned.

@chubin chubin changed the title Error With Location Fetching when no location is passed Improve IP-based geolocation Mar 23, 2025
@tr1xem
Copy link
Author

tr1xem commented Mar 24, 2025

Tuned as ever

Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Projects
None yet
Development

No branches or pull requests

2 participants