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

[Verification/Geofencing]: Decrease the minimum of a Circle radius #282

Open
maxl2287 opened this issue Dec 17, 2024 · 3 comments · May be fixed by #285
Open

[Verification/Geofencing]: Decrease the minimum of a Circle radius #282

maxl2287 opened this issue Dec 17, 2024 · 3 comments · May be fixed by #285
Assignees
Labels
enhancement New feature or request

Comments

@maxl2287
Copy link
Contributor

Problem description
As of now the minimum of a radius is set to 2000 meters, which is quite too large as a minimum.
We discovered the issue as we wanted to test a geofencing creation inside of a building, but we were forced to use a geofence with a minimum of 2 kilometers.

Possible evolution
Decrease the minimum to a smaller value, e.g. 50 meters

@maxl2287 maxl2287 added the enhancement New feature or request label Dec 17, 2024
@alpaycetin74
Copy link
Collaborator

This was probably a result of privacy concerns. The minimum in the API specification could be even much smaller, but the implementer must be made aware of legal implications and define a minimum for their use case.

@bigludo7
Copy link
Collaborator

I'm fine to shift from 2000 to 1 but we need to add a note in the documentation part of the API specification to clearly indicate that the area surface could be restricted locally depending on regulation (eg. the yaml specify a minimum of 1 but an implementation could restrict to a minimum of 1000).

For location-verification we probably need to define an error to manage this case (an implementation restricts the value above 1000 and got 150 in the request). We can use 422 + AREA_NOT_COVERED (as defined in geofencing).

@jlurien
Copy link
Collaborator

jlurien commented Dec 26, 2024

I agree, that we need to indicate that implementations may consider a minimum radius as valid input, due to privacy or implementation constraints, and as @bigludo7 proposes, define a specific error to signal that the input area is not supported,

Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
enhancement New feature or request
Projects
None yet
4 participants