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

Using the IP from the request for the GeoIP endpoint #10699

Closed
teolemon opened this issue Aug 14, 2024 · 0 comments · Fixed by #10700 or #10714
Closed

Using the IP from the request for the GeoIP endpoint #10699

teolemon opened this issue Aug 14, 2024 · 0 comments · Fixed by #10700 or #10714

Comments

@teolemon
Copy link
Member

          Thanks for the PR.

However, getting the public IP from Dart is impossible without an external service (ex : IPify).
Would it be possible to change the endpoint from:
https://world.openfoodfacts.net/api/v3/geoip/85.214.132.117
to
https://world.openfoodfacts.net/api/v3/geoip/

Basically, if the IP is missing, use the one from the request?

Originally posted by @g123k in #10594 (comment)

stephanegigandet pushed a commit that referenced this issue Aug 22, 2024
🤖 I have created a release *beep* *boop*
---


##
[2.42.0](v2.41.0...v2.42.0)
(2024-08-22)


### Features

* geoip/ api use the remote address if ip not provided as param
([#10700](#10700))
([7c1e3f2](7c1e3f2)),
closes
[#10699](#10699)
* knowledge API to specify which panels should be returned
([#10701](#10701))
([384961a](384961a))
* migration script to batch import existing orgs in CRM
([#10683](#10683))
([1801b48](1801b48))


### Bug Fixes

* remove robotoff webhook trigger
([#10723](#10723))
([47e1174](47e1174))
* remove unused jquery-ui safe-active-element module
([#10708](#10708))
([9407e86](9407e86))
* Synonyms in taxonomized suggestions
([#10722](#10722))
([3419a1e](3419a1e))

---
This PR was generated with [Release
Please](https://github.com/googleapis/release-please). See
[documentation](https://github.com/googleapis/release-please#release-please).
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Projects
1 participant