{"action":"created","issue":{"url":"https://api.github.com/repos/httpwg/http-extensions/issues/700","repository_url":"https://api.github.com/repos/httpwg/http-extensions","labels_url":"https://api.github.com/repos/httpwg/http-extensions/issues/700/labels{/name}","comments_url":"https://api.github.com/repos/httpwg/http-extensions/issues/700/comments","events_url":"https://api.github.com/repos/httpwg/http-extensions/issues/700/events","html_url":"https://github.com/httpwg/http-extensions/issues/700","id":365172083,"node_id":"MDU6SXNzdWUzNjUxNzIwODM=","number":700,"title":"Clarify behavior for multiple values for CH headers","user":{"login":"igrigorik","id":10652,"node_id":"MDQ6VXNlcjEwNjUy","avatar_url":"https://avatars0.githubusercontent.com/u/10652?v=4","gravatar_id":"","url":"https://api.github.com/users/igrigorik","html_url":"https://github.com/igrigorik","followers_url":"https://api.github.com/users/igrigorik/followers","following_url":"https://api.github.com/users/igrigorik/following{/other_user}","gists_url":"https://api.github.com/users/igrigorik/gists{/gist_id}","starred_url":"https://api.github.com/users/igrigorik/starred{/owner}{/repo}","subscriptions_url":"https://api.github.com/users/igrigorik/subscriptions","organizations_url":"https://api.github.com/users/igrigorik/orgs","repos_url":"https://api.github.com/users/igrigorik/repos","events_url":"https://api.github.com/users/igrigorik/events{/privacy}","received_events_url":"https://api.github.com/users/igrigorik/received_events","type":"User","site_admin":false},"labels":[{"id":290789441,"node_id":"MDU6TGFiZWwyOTA3ODk0NDE=","url":"https://api.github.com/repos/httpwg/http-extensions/labels/client-hints","name":"client-hints","color":"fbca04","default":false}],"state":"open","locked":false,"assignee":{"login":"igrigorik","id":10652,"node_id":"MDQ6VXNlcjEwNjUy","avatar_url":"https://avatars0.githubusercontent.com/u/10652?v=4","gravatar_id":"","url":"https://api.github.com/users/igrigorik","html_url":"https://github.com/igrigorik","followers_url":"https://api.github.com/users/igrigorik/followers","following_url":"https://api.github.com/users/igrigorik/following{/other_user}","gists_url":"https://api.github.com/users/igrigorik/gists{/gist_id}","starred_url":"https://api.github.com/users/igrigorik/starred{/owner}{/repo}","subscriptions_url":"https://api.github.com/users/igrigorik/subscriptions","organizations_url":"https://api.github.com/users/igrigorik/orgs","repos_url":"https://api.github.com/users/igrigorik/repos","events_url":"https://api.github.com/users/igrigorik/events{/privacy}","received_events_url":"https://api.github.com/users/igrigorik/received_events","type":"User","site_admin":false},"assignees":[{"login":"igrigorik","id":10652,"node_id":"MDQ6VXNlcjEwNjUy","avatar_url":"https://avatars0.githubusercontent.com/u/10652?v=4","gravatar_id":"","url":"https://api.github.com/users/igrigorik","html_url":"https://github.com/igrigorik","followers_url":"https://api.github.com/users/igrigorik/followers","following_url":"https://api.github.com/users/igrigorik/following{/other_user}","gists_url":"https://api.github.com/users/igrigorik/gists{/gist_id}","starred_url":"https://api.github.com/users/igrigorik/starred{/owner}{/repo}","subscriptions_url":"https://api.github.com/users/igrigorik/subscriptions","organizations_url":"https://api.github.com/users/igrigorik/orgs","repos_url":"https://api.github.com/users/igrigorik/repos","events_url":"https://api.github.com/users/igrigorik/events{/privacy}","received_events_url":"https://api.github.com/users/igrigorik/received_events","type":"User","site_admin":false}],"milestone":null,"comments":11,"created_at":"2018-09-29T22:28:31Z","updated_at":"2018-11-03T03:09:58Z","closed_at":null,"author_association":"MEMBER","body":"Migrating [feedback from Kari Hurtta](https://lists.w3.org/Archives/Public/ietf-http-wg/2018JulSep/0137.html).. \r\n\r\n> On Wed, Jul 25, 2018 at 9:14 AM HTTP Working Group <ietf-http-wg@w3.org>\r\n> wrote:\r\n> \r\n> > |   If DPR occurs in a message more than once, the last value overrides\r\n> > |   all previous occurrences.\r\n> \r\n> So is that saying that if there is several DPR values on\r\n> > DPR Header Field, last value is used ?\r\n> >\r\n> \r\n> Yes, that's the intent.\r\n> \r\n> If that in interpreted only that if last DPR header field is used,\r\n> > and header field is ignored if it does not fit\r\n> >\r\n> >    DPR = 1*DIGIT [ \".\" 1*DIGIT ]\r\n> >\r\n> > syntax, result is different is recipient combines multiple\r\n> > DPR headers.\r\n> \r\n> \r\n> The intent was to specify that last valid value (regardless if it is within\r\n> a combined or separate header) is the one that should be used by the\r\n> client. My read of the current wording is that it matches this intent, but\r\n> I'll defer to the experts here on how to best word this if that's not the\r\n> case.\r\n> \r\n> Mark, Julian: any guidance on this one?\r\n> \r\n> \r\n> > |   If <X> occurs in a message more than once, the last value\r\n> > |   overrides all previous occurrences.\r\n> >\r\n> > So it should mention that on these header fields also last value\r\n> > is used when values are ',' -separated.\r\n> >\r\n> \r\n> My understanding is that this is implicit, but if that's not the case,\r\n> happy to rework the wording.\r\n\r\n--- \r\n\r\nJulian:\r\n> No, it's not implicit, so yes, a clarification would be good. I don't think we have good text yet."},"comment":{"url":"https://api.github.com/repos/httpwg/http-extensions/issues/comments/435555806","html_url":"https://github.com/httpwg/http-extensions/issues/700#issuecomment-435555806","issue_url":"https://api.github.com/repos/httpwg/http-extensions/issues/700","id":435555806,"node_id":"MDEyOklzc3VlQ29tbWVudDQzNTU1NTgwNg==","user":{"login":"mnot","id":74384,"node_id":"MDQ6VXNlcjc0Mzg0","avatar_url":"https://avatars1.githubusercontent.com/u/74384?v=4","gravatar_id":"","url":"https://api.github.com/users/mnot","html_url":"https://github.com/mnot","followers_url":"https://api.github.com/users/mnot/followers","following_url":"https://api.github.com/users/mnot/following{/other_user}","gists_url":"https://api.github.com/users/mnot/gists{/gist_id}","starred_url":"https://api.github.com/users/mnot/starred{/owner}{/repo}","subscriptions_url":"https://api.github.com/users/mnot/subscriptions","organizations_url":"https://api.github.com/users/mnot/orgs","repos_url":"https://api.github.com/users/mnot/repos","events_url":"https://api.github.com/users/mnot/events{/privacy}","received_events_url":"https://api.github.com/users/mnot/received_events","type":"User","site_admin":false},"created_at":"2018-11-03T03:09:58Z","updated_at":"2018-11-03T03:09:58Z","author_association":"MEMBER","body":"> The last valid value of the aggregated DPR field value overrides all previous occurrences.\r\n\r\nThe issue is that it's defined to only have one value. I think you'd need to say something like:\r\n\r\n> if more than one DPR header field is present (either because multiple header instances exist, or because they've been folded into comma-separated values)...\r\n\r\nOr, to *really* define it, define an algorithm."}}
{"actor":{"display_login":"mnot"}}