#204: saveData attribute in Network Information API

Visit on Github.

Opened Oct 6, 2017

Dear Sirs and Madam of TAG!

I'm requesting a TAG review of:

Further details (optional):

  • Relevant time constraints or deadlines: We'd like to ship in Chrome M64
  • I have reviewed the TAG's API Design Principles

We'd prefer the TAG provide feedback as (please select one):

  • open issues in our Github repo for each point of feedback
  • open a single issue in our Github repo for the entire review
  • leave review feedback as a comment in this issue and @-notify [github usernames]

Discussions

Comment by @slightlyoff Oct 31, 2017 (See Github)

Overall I like this. Having this available at the network level seems great.

I do continue to be worried about the design of Client Hints overall; not having this information in the intial request hobbles what servers can reasonably do, inducing the to cookie users or create redirects for the purposes of just getting this information.

Comment by @torgo Nov 29, 2017 (See Github)

@slightlyoff "anything you should be able to see at the network level that is not security sensitive should be reflected back to page content - you should not have to go through round-trips. Same data is being sent through client hints header. This tells javascript code the same thing the server could have learned. So :+1:"

Comment by @dbaron Nov 29, 2017 (See Github)

It's a little worrying that http://httpwg.org/http-extensions/client-hints.html#save-data and https://wicg.github.io/netinfo/#-dfn-savedata-dfn-attribute seem to be specifying the same material. (Or are they?) Seems like one spec should be canonical for each thing -- but if the save data features are split between two specs they should definitely point to each other clearly.

Comment by @torgo Nov 29, 2017 (See Github)

Some concerns from @cynthia on privacy considerations.

Comment by @igrigorik Dec 6, 2017 (See Github)

@dbaron we moved Save-Data header definition from IETF doc into NetInfo spec in recent update. New IETF draft hasn't been published yet, hence the duplication.. but that won't be the case soon.

Discussed Jan 1, 2018 (See Github)

Sanwghan: I had something about privacy I wanted to add in the review....

Alex: I am a fan and want to see it move foreward.

Sangwhan: I will drop the comments in github. Can we revisit next week?

--

Comment by @torgo Jan 16, 2018 (See Github)

Some discussion on this on today's call.

Comment by @torgo Jan 16, 2018 (See Github)

Agreed to discuss at f2f and hopefully we can get you on the phone for that discussion @igrigorik. Will be Jan 31 - Feb 2 (London).

Comment by @davidmurdoch Jan 24, 2018 (See Github)

Will this property be exposed to CSS media queries? I'd love to be able to do:

<picture>
    <source type="image/webp" srcset="/img-low-quality.webp" media="(save-data: on)">
    <source type="image/webp" srcset="/img.webp">
    <img src="/img.jpg">
</picture>
Comment by @torgo Feb 2, 2018 (See Github)

Discussed at London F2F day 3.

Comment by @triblondon Feb 2, 2018 (See Github)

TAG review comments:

  • Fingerprinting potential by correlating patterns of savedata status across multiple origins (ie I will exhibit the same pattern of moving from good to poor service areas on all websites I am visiting over the same period)
    • Meta-point: dynamic hint variables can provide trackable profiles when monitored over time.
    • -> Seems to be addressed in privacy section of the spec
  • Not clear whether this is informed by exactly the same data source as the CH header, and we feel they should be the same spec if they are the same thing.
Comment by @triblondon Feb 2, 2018 (See Github)

TAG review conclusion:

  • Fingerprinting issues are still a source of concern, even given the text in the spec. We note that the WebRTC vulnerability identified in the spec has been fixed following previous TAG feedback to that group.
  • Some TAG members take the view that the considerations given to privacy are not adequate and that the group should consider further work on mitigating the fingerprinting potential, but we do not have consensus on this.
Comment by @Malvoz Jun 18, 2018 (See Github)

@davidmurdoch

Will this property be exposed to CSS media queries

There is a proposal in https://github.com/w3c/csswg-drafts/issues/2370.