Skip to content

Commit 0e04901

Browse files
committed
feat($interpolate): escaped interpolation expressions
This CL enables interpolation expressions to be escaped, by prefixing each character of their start/end markers with a REVERSE SOLIDUS U+005C, and to render the escaped expression as a regular interpolation expression. Example: `<span ng-init="foo='Hello'">{{foo}}, \\{\\{World!\\}\\}</span>` would be rendered as: `<span ng-init="foo='Hello'">Hello, {{World!}}</span>` This will also work with custom interpolation markers, for example: module. config(function($interpolateProvider) { $interpolateProvider.startSymbol('\\\\'); $interpolateProvider.endSymbol('//'); }). run(function($interpolate) { // Will alert with "hello\\bar//": alert($interpolate('\\\\foo//\\\\\\\\bar\\/\\/')({foo: "hello", bar: "world"})); }); This change effectively only changes the rendering of these escaped markers, because they are not context-aware, and are incapable of preventing nested expressions within those escaped markers from being evaluated. Therefore, backends are encouraged to ensure that when escaping expressions for security reasons, every single instance of a start or end marker have each of its characters prefixed with a backslash (REVERSE SOLIDUS, U+005C) Closes angular#5601
1 parent 95cdb53 commit 0e04901

File tree

2 files changed

+54
-1
lines changed

2 files changed

+54
-1
lines changed

src/ng/interpolate.js

+13-1
Original file line numberDiff line numberDiff line change
@@ -81,7 +81,13 @@ function $InterpolateProvider() {
8181

8282
this.$get = ['$parse', '$exceptionHandler', '$sce', function($parse, $exceptionHandler, $sce) {
8383
var startSymbolLength = startSymbol.length,
84-
endSymbolLength = endSymbol.length;
84+
endSymbolLength = endSymbol.length,
85+
escapedStartRegexp = new RegExp(startSymbol.replace(/./g, escape), 'g'),
86+
escapedEndRegexp = new RegExp(endSymbol.replace(/./g, escape), 'g');
87+
88+
function escape(ch) {
89+
return '\\\\\\' + ch;
90+
}
8591

8692
/**
8793
* @ngdoc service
@@ -176,6 +182,12 @@ function $InterpolateProvider() {
176182
}
177183
}
178184

185+
forEach(separators, function(key, i) {
186+
separators[i] = separators[i].
187+
replace(escapedStartRegexp, startSymbol).
188+
replace(escapedEndRegexp, endSymbol);
189+
});
190+
179191
if (separators.length === expressions.length) {
180192
separators.push('');
181193
}

test/ng/interpolateSpec.js

+41
Original file line numberDiff line numberDiff line change
@@ -61,6 +61,47 @@ describe('$interpolate', function() {
6161
}));
6262

6363

64+
describe('interpolation escaping', function() {
65+
var obj;
66+
67+
beforeEach(function() {
68+
obj = {foo: 'Hello', bar: 'World'};
69+
});
70+
71+
it('should support escaping interpolation signs', inject(function($interpolate) {
72+
expect($interpolate('{{foo}} \\{\\{bar\\}\\}')(obj)).toBe('Hello {{bar}}');
73+
expect($interpolate('\\{\\{foo\\}\\} {{bar}}')(obj)).toBe('{{foo}} World');
74+
}));
75+
76+
77+
it('should unescape multiple expressions', inject(function($interpolate) {
78+
expect($interpolate('\\{\\{foo\\}\\}\\{\\{bar\\}\\} {{foo}}')(obj)).toBe('{{foo}}{{bar}} Hello');
79+
expect($interpolate('{{foo}}\\{\\{foo\\}\\}\\{\\{bar\\}\\}')(obj)).toBe('Hello{{foo}}{{bar}}');
80+
expect($interpolate('\\{\\{foo\\}\\}{{foo}}\\{\\{bar\\}\\}')(obj)).toBe('{{foo}}Hello{{bar}}');
81+
expect($interpolate('{{foo}}\\{\\{foo\\}\\}{{bar}}\\{\\{bar\\}\\}{{foo}}')(obj)).toBe('Hello{{foo}}World{{bar}}Hello');
82+
}));
83+
84+
85+
it('should support customizing escape signs', function() {
86+
module(function($interpolateProvider) {
87+
$interpolateProvider.startSymbol('[[');
88+
$interpolateProvider.endSymbol(']]');
89+
});
90+
inject(function($interpolate) {
91+
expect($interpolate('[[foo]] \\[\\[bar\\]\\]')(obj)).toBe('Hello [[bar]]');
92+
});
93+
});
94+
95+
96+
it('should unescape incomplete escaped expressions', inject(function($interpolate) {
97+
expect($interpolate('\\{\\{foo{{foo}}')(obj)).toBe('{{fooHello');
98+
expect($interpolate('\\}\\}foo{{foo}}')(obj)).toBe('}}fooHello');
99+
expect($interpolate('foo{{foo}}\\{\\{')(obj)).toBe('fooHello{{');
100+
expect($interpolate('foo{{foo}}\\}\\}')(obj)).toBe('fooHello}}');
101+
}));
102+
});
103+
104+
64105
describe('interpolating in a trusted context', function() {
65106
var sce;
66107
beforeEach(function() {

0 commit comments

Comments
 (0)