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

How to specify desired accuracy / resolution of data? #49

Open
pes10k opened this issue Jun 30, 2020 · 19 comments
Open

How to specify desired accuracy / resolution of data? #49

pes10k opened this issue Jun 30, 2020 · 19 comments
Assignees
Labels
privacy-tracker Group bringing to attention of Privacy, or tracked by the Privacy Group but not needing response. security-tracker Group bringing to attention of security, or tracked by the security Group but not needing response.

Comments

@pes10k
Copy link

pes10k commented Jun 30, 2020

The spec currently says that

Section 7.2 Requirements

The Geolocation API MUST allow an application to specify a desired accuracy level of the location information.

This is a terrific idea, and enables all sorts of good practices (i might just want to know what city someone lives, but not their street, etc, for ethical, safety and legal reasons).

However, I don't see a way to do this in the API. I see the enableHighAccuracy flag, but i) this is a boolean option, and ii) its not clear to the caller what this will mean, or what the resolution of true or false will be here. It would be a terrific to have a way of allowing sites to say "no more accurate than…" or "only city level" or "only use IP based location" etc.

Basically, the ask here to allow sites to blind themselves from non-needed location data

@pes10k pes10k added the privacy-tracker Group bringing to attention of Privacy, or tracked by the Privacy Group but not needing response. label Jun 30, 2020
@marcoscaceres
Copy link
Member

Note, although useful, we are updating a REC to reflect updated implementations, not add new features. This is probably out of scope for now.

@pes10k
Copy link
Author

pes10k commented Jul 1, 2020

Yep! Didn't mean this one as any kind of blocking issue. Definitely feature request level issue.

Though (and its a super minor thing) i think the text in the requirements section about "specify a desired accuracy level" seems a bit at odds with the rest of the document. But any change requested here would be text-tweak level, not feature request level

@anssiko
Copy link
Member

anssiko commented Jul 1, 2020

As @marcoscaceres rightly points out, new feature proposals beyond editorial text tweaking are out of scope for the Geolocation API since this spec is in maintenance mode. Maintenance includes hardening this quite old API for privacy and security to the extend we can without breaking existing content.

For new feature discussion, there's another API called Geolocation Sensor. Please see its issues labeled geo-accuracy and submit this feedback there. See also all open issues.

Related, and probably of interest to @pes10k as well:

Feedback is also welcome on the new use cases that motivate every new geolocation feature under consideration. Out of these use cases, especially the background operations are highly demanded by web developers but at the same time they are also the most tough ones to solve in a privacy-preserving manner and thus have not advanced beyond use case solicitation at this point. We're hopeful we'll get there eventually, but we don't yet know what API shape and form will get us there. We think, however, that retrofitting the Geolocation API with new features for use cases such as background operation is not a viable option due to its aging design.

@pes10k
Copy link
Author

pes10k commented Jul 1, 2020

Thank you both for your notes above. I'm holding off adding comments to Geolocation Sensor since my understanding is the relevant WG charter has objections (including regarding geolocation). If those get resolved, or if ive misunderstood the situation, i'll happily move this issue over there.

@anssiko
Copy link
Member

anssiko commented Jul 1, 2020

@pes10k wrote:

Thank you both for your notes above. I'm holding off adding comments to Geolocation Sensor since my understanding is the relevant WG charter has objections (including regarding geolocation). If those get resolved, or if ive misunderstood the situation, i'll happily move this issue over there.

@pes10k I think you may have slightly misunderstood how the W3C Process works. Let me try to clarify.

First, The Director has not announced any changes regarding this WG or the Geolocation Sensor work, so please feel free to submit your feedback to the https://github.com/w3c/geolocation-sensor repo where the context for your proposal is. In particular, please review the issues labeled with geo-accuracy and chime in.

Second, the WG is not aware of any technical arguments against the Geolocation Sensor spec that'd suggest changes to the direction. The WG acknowledges the historical context and that the problem is not easy to solve, but that's not a valid reason to disinvest. This, especially given the strong demand for the new geolocation features from the web developer community. As per the WG's commitment, we solve also this problem in a secure and privacy-preserving manner. As a WG, we also benefit from horizontal review from experts in privacy, security, others. Of course, we're lucky to have our resident privacy experts in this WG who are scrutinizing the WG's work closely. And we have you watching us!

Last but not least, the fact that you opened this issue adds to the pile of evidence we need to keep the Geolocation Sensor as its own separate WG deliverable as to allow discussion and development of new features for geolocation without interfering with the Geolocation API maintenance effort that is also important, but has its own timelines and priorities. These two APIs coexist.

I'll ping @samuelweiler to ensure this valuable feedback from @pes10k is considered by the W3M.

@pes10k
Copy link
Author

pes10k commented Jul 1, 2020

@anssiko I appreciate your comments above, but there as been, i believe, at least a few AC comments asking the "Devices and Sensors Working Group" to cease work on the geolocation spec, and for it to stay in this WG. I'd like to wait for the resolution of that conversation before moving comments over. Either way, i will make sure this issue winds up in one (or both) repos

@anssiko
Copy link
Member

anssiko commented Jul 1, 2020

@pes10k wrote:

but there as been, i believe, at least a few AC comments asking the "Devices and Sensors Working Group" to cease work on the geolocation spec, and for it to stay in this WG

Given "Devices and Sensors Working Group" == "this WG", I feel obligated to clarify one more thing: both the Geolocation API (since March 2019) and Geolocation Sensor (since June 2018) are deliverables of the Devices and Sensors Working Group.

Geolocation Working Group that produced an initial version of the Geolocation API closed in May 2017 leaving that spec in an unmaintained state. Devices and Sensors Working Group was concerned about that and adopted the Geolocation API spec to continue its maintenance in parallel to the new feature work in the Geolocation Sensor spec it had started earlier.

@samuelweiler
Copy link
Member

samuelweiler commented Jul 2, 2020

I'm going to take this thread in a different direction: it seems odd to have this spec list some requirements (with 2119 MUSTs, even) that are then not met - or very weakly met (e.g. with a single bit?!?!?). A casual reviewer might take solace from reading the requirements, even though they don't reflect the reality of the spec.

I suggest adding a note to this requirements section saying that the spec does not meet requirement #7 (or does a poor job of meeting it). Ideally, the WG would consider each of the other items also.

I think this suggestion falls into the realm of text tweaking rather than new features.

@samuelweiler
Copy link
Member

samuelweiler commented Aug 11, 2020

Our security reviewer suggested this feature in #57 and separately called out the weird bit about having 2119 language in a non-normative section #56 without commenting on how they tie together, as I observed on July 2nd..

I'm flagging this for resolution.

@marcoscaceres
Copy link
Member

Closed via b1fac55

@marcoscaceres
Copy link
Member

oops, closed the wrong bug.

@marcoscaceres marcoscaceres reopened this Apr 2, 2021
@marcoscaceres
Copy link
Member

Clarified that, “And through enableHighAccuracy, supports requesting "high accuracy" position data, though the request can be ignored by the user agent.”

In the introduction.
https://w3c.github.io/geolocation-api/#introduction

@samuelweiler
Copy link
Member

Looking back on this, it looks like the requirements text I comment on above has been entirely removed.

That's somewhat unsatisfying. As @pes10k writes, the requirement spoke to a useful feature; "This is a terrific idea, and enables all sorts of good practices (i might just want to know what city someone lives, but not their street, etc, for ethical, safety and legal reasons)."

I think the WG wants to not add any functionality in this space? Reopening this to confirm the WG intent. Should we file this as a feature request on a future spec?

@anssiko
Copy link
Member

anssiko commented Dec 14, 2021

I think the WG wants to not add any functionality in this space? Reopening this to confirm the WG intent. Should we file this as a feature request on a future spec?

Per https://www.w3.org/2020/11/das-wg-charter.html the Geolocation API is in maintenance and as such the focus of the WG is to specify what has been implemented and move the Geolocation API with that feature set toward REC. @marcoscaceres has been driving this effort as an editor, working with implementers to ensure what is specified is also implemented.

New features and functionality have been discussed in the context of the Geolocation Sensor spec based on the Generic Sensor API. Please review the issues labeled with geo-accuracy and chime in there.

@marcoscaceres
Copy link
Member

marcoscaceres commented Dec 14, 2021

Echoing what @anssiko said. This is probably in scope for the updated REC. But definately something we can explore after, as this will be an "updatable rec". It would need implementer buy-in tho.

@marcoscaceres
Copy link
Member

@samuelweiler, please see #49 (comment) and my comment above. We can keep this issue open, but would like a resolution that it will either be part of Geolocation Sensor or something we work on after REC.

@pes10k
Copy link
Author

pes10k commented Jan 31, 2022

Just following up here after figuring out who PING side should follow up. PING-side, we're fine closing this as long as the WG agrees to take it up post REC

@marcoscaceres
Copy link
Member

@pes10k, yep, we agree to take it up later. Let's leave it open.

@samuelweiler, does it need to be resolved over on some other (security) tracker? (i.e., the "security-needs-resolution" label)

@samuelweiler samuelweiler added security-tracker Group bringing to attention of security, or tracked by the security Group but not needing response. and removed security-needs-resolution Issue the security Group has raised and looks for a response on. labels Feb 1, 2022
@samuelweiler
Copy link
Member

Thanks, @marcoscaceres. I did some tracker cleanup.

Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
privacy-tracker Group bringing to attention of Privacy, or tracked by the Privacy Group but not needing response. security-tracker Group bringing to attention of security, or tracked by the security Group but not needing response.
Projects
None yet
Development

No branches or pull requests

4 participants