Skip to content

Commit

Permalink
feat(security): categorize <track src> as a regular URL.
Browse files Browse the repository at this point in the history
After security review, it turns out we were too paranoid about <track src>. Its content is not actually active or dangerous.

Fixes angular#10089.
  • Loading branch information
mprobst committed Jul 21, 2016
1 parent 76b8a49 commit a441b5b
Showing 1 changed file with 4 additions and 5 deletions.
9 changes: 4 additions & 5 deletions modules/@angular/compiler/src/schema/dom_security_schema.ts
Original file line number Diff line number Diff line change
Expand Up @@ -36,10 +36,10 @@ registerContext(SecurityContext.HTML, [
registerContext(SecurityContext.STYLE, ['*|style']);
// NB: no SCRIPT contexts here, they are never allowed due to the parser stripping them.
registerContext(SecurityContext.URL, [
'*|formAction', 'area|href', 'area|ping', 'audio|src', 'a|href',
'a|ping', 'blockquote|cite', 'body|background', 'del|cite', 'form|action',
'img|src', 'img|srcset', 'input|src', 'ins|cite', 'q|cite',
'source|src', 'source|srcset', 'video|poster', 'video|src',
'*|formAction', 'area|href', 'area|ping', 'audio|src', 'a|href',
'a|ping', 'blockquote|cite', 'body|background', 'del|cite', 'form|action',
'img|src', 'img|srcset', 'input|src', 'ins|cite', 'q|cite',
'source|src', 'source|srcset', 'track|src', 'video|poster', 'video|src',
]);
registerContext(SecurityContext.RESOURCE_URL, [
'applet|code',
Expand All @@ -55,5 +55,4 @@ registerContext(SecurityContext.RESOURCE_URL, [
'object|codebase',
'object|data',
'script|src',
'track|src',
]);

0 comments on commit a441b5b

Please sign in to comment.