search

w3c / sensors

Generic Sensor API
https://www.w3.org/TR/generic-sensor/
Other
127 stars 59 forks source link

Update "Security and privacy considerations" and "Permissions API" #400

Closed anssiko closed 4 years ago

anssiko commented 4 years ago

These changes address wide review feedback from PING.

Fix #396

PTAL @snyderp

:boom: Error: 403 Forbidden :boom:

PR Preview failed to build. (Last tried on Nov 14, 2019, 9:21 AM UTC).

More PR Preview relies on a number of web services to run. There seems to be an issue with the following one: :rotating_light: [HTML Diff Service](http://services.w3.org/htmldiff) - The HTML Diff Service is used to create HTML diffs of the spec changes suggested in a pull request. :link: [Related URL](https://services.w3.org/htmldiff?doc1=https%3A%2F%2Fpr-preview.s3.amazonaws.com%2Fw3c%2Fsensors%2Fpull%2F400%2F89c2080.html&doc2=https%3A%2F%2Fpr-preview.s3.amazonaws.com%2Fw3c%2Fsensors%2Fpull%2F400.html) ``` HTML Diff service

W3C W3C Editors homepage

Create Diff between HTML pages

An error (403 Forbidden) occured trying to get https://pr-preview.s3.amazonaws.com/w3c/sensors/pull/400/89c2080.html.

Address of reference document:

Address of new document:

Tip: if the document uses the W3C convention on linking to its previous version, you can specify only the address of the new document — the previous link will be automatically detected.

Diff markings

This service relies on GNU diff. The found differences are roughly marked as follow:

  • deleted text is shown in pink with down-arrows (as styled for a <del> element)
  • where there is replacement, it’s shown in green with bi-directional arrows,
  • where there is newly inserted text, it’s yellow with up arrows (<ins> element)
script $Revision$ of $Date$
by Dominique Hazaël-Massieux
based on Shane McCarron’ Perl script wrapped in a Python CGI
``` _If you don't have enough information above to solve the error by yourself (or to understand to which web service the error is related to, if any), please [file an issue](https://github.com/tobie/pr-preview/issues/new?title=Error%20not%20surfaced%20properly&body=See%20w3c/sensors%23400.)._
anssiko commented 4 years ago

(Please ignore the PR Preview build fail for now, and just view the .bs source. @xfq to investigate.)

xfq commented 4 years ago

Informal LGTM from me.

anssiko commented 4 years ago

@snyderp please review this PR at your earliest opportunity so that it makes it to the revised Candidate Recommendation publication.

pes10k commented 4 years ago

@anssiko I am not 100% sure i understand how to interpret this. Is the intent to say sensor's should be treated with more care than other permissioned APIs, exactly the same, or that its not this standard's place to say, and its all up to the UA?

anssiko commented 4 years ago

These proposed changes modify non-normative text only, or conversely, do not change any normative requirements for implementers.

pes10k commented 4 years ago

@anssiko thanks, I understand. I agree that this PR fixes https://github.com/w3c/sensors/issues/396