add note on qweb template safety

This commit is contained in:
Géry Debongnie
2019-01-28 11:55:05 +01:00
parent 250f82b9ca
commit 3906c7eee1
+4
View File
@@ -167,6 +167,10 @@ We have 3 main folders and 3 main files:
- **Qweb**
- check qweb tests and see if it is reasonable (escaping? safety/security?)
- Expression evaluation: make sure we cannot inject JS in template function.
For example, a t-props directive such as t-props="state.someValueFromServer"
where the value would be something such as `"};console.log('Owned': context.env.actionManager.doAction('fuck you'))`
- remove the "if (${exprID} || ${exprID} === 0) {"
- need to implement t-extend (?)
- improve qweb generated code: do not assign object/array if no props/attrs/children