-
Notifications
You must be signed in to change notification settings - Fork 17
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
Defer to TTML2 for security and privacy considerations #307
Conversation
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
This change is significant in that it both removes any IMSC-specific text and imports all of the security and privacy considerations of TTML2, even those that do not apply.
Happy to discuss this on the call this week, but it seems to me that we need to:
-
Call out the security and privacy implications of extension features added in this specification, to which the TTML2 considerations by definition do not apply
-
Call out any considerations listed in TTML2 that do not apply here because they are profiled out, for example the one about hyperlinking mechanisms.
@nigelmegitt I see no benefit in spending time subsetting the TTM2 privacy and security considerations. Can you list them? |
@palemieux You mean you don't want to and you're asking me to do it? |
@nigelmegitt Oh no. I mean I do not think it is worth anyone doing it :) |
@palemieux I disagree - if we know that a consideration does not apply, we should clarify that: it is a potential benefit of using the profile. |
@nigelmegitt Alright. Please go ahead then. |
As discussed in the TTWG call just now the following TTML2 considerations do not apply to IMSC 1.1:
So add text such as: _With the exception of the following considerations the privacy and security considerations of TTML2 apply. Those exceptions are:
|
imsc1/spec/ttml-ww-profiles.html
Outdated
<ul> | ||
<li>Appendix P.7 (Access to Processing State), since the condition attribute is not supported;</li> | ||
|
||
<li>Appendix P.8 (Hyperlinking Mechanisms), since the xlink linking mechanism is not supported.</li> |
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
Happy with the words - do we need to format the terms condition
and xlink
to look like code?
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
Looks good to me.
…curity-privacy-to-ttml2
Closes #280