w3c / ttml2

Timed Text Markup Language 2 (TTML2)
https://w3c.github.io/ttml2/
Other
41 stars 16 forks source link

Draft language to address font fingerprinting mitigation (#1202). #1210

Closed skynavga closed 3 years ago

skynavga commented 4 years ago

Closes #1202.

css-meeting-bot commented 3 years ago

The Timed Text Working Group just discussed Draft language to address font fingerprinting mitigation (#1202). w3c/ttml2#1210, and agreed to the following:

The full IRC log of that discussion <nigel_> Topic: Draft language to address font fingerprinting mitigation (#1202). w3c/ttml2#1210
<nigel_> github: https://github.com/w3c/ttml2/pull/1210
<nigel_> Nigel: I reviewed this (opened since July), and think it is an improvement and a step on the way but maybe not the end of the changes we need.
<nigel_> Andreas: I added one comment to the pull request where the addition is to strongly recommend not to dereference external fonts.
<nigel_> .. In the current pull request it says "should consider not dereferencing"
<nigel_> .. I think the "consider" should be removed.
<nigel_> .. The reasoning is that we had a long discussion with PING, who asked for more, they wanted it normative.
<nigel_> .. It is now strong language in a non-normative section.
<nigel_> .. I think we should not weaken it more, and it would be better to say "should not do it".
<nigel_> Nigel: I think Glenn already indicated he would accept it, and I certainly would.
<nigel_> Glenn: I don't like it but I could live with it.
<nigel_> Nigel: I can't see Andreas's comment on the pull request, only my proposal.
<nigel_> Andreas: I commented it but I maybe need to complete the review.
<nigel_> Nigel: If we make that change then my change would not be needed.
<nigel_> .. I would like to merge this - any requests for more time to review?
<nigel_> group: [no requests for more time]
<nigel_> Nigel: In that case when Andreas's change has been processed we should be good to merge.
<nigel_> SUMMARY: Andreas's proposal to be applied
skynavga commented 3 years ago

Merge per request from @nigelmegitt in order to advance to new CR.