aboutsummaryrefslogtreecommitdiffhomepage
path: root/tpl/page.header.html
diff options
context:
space:
mode:
authorArthurHoaro <arthur@hoa.ro>2015-06-11 13:53:27 +0200
committerArthurHoaro <arthur@hoa.ro>2015-06-23 16:35:36 +0200
commit5f85fcd863fe261921953ea3bd1742f3e1b7cf68 (patch)
tree5615922c1c696ec04cc60625a8d401b2b297a462 /tpl/page.header.html
parent0923a2bc1b097bf1def882722db489d83d95c423 (diff)
downloadShaarli-5f85fcd863fe261921953ea3bd1742f3e1b7cf68.tar.gz
Shaarli-5f85fcd863fe261921953ea3bd1742f3e1b7cf68.tar.zst
Shaarli-5f85fcd863fe261921953ea3bd1742f3e1b7cf68.zip
Working on shaarli/Shaarli#224
I reviewed character escaping everywhere with the following ideas: * use a single common function to escape user data: `escape` using `htmlspecialchars`. * sanitize fields in `index.php` after reading them from datastore and before sending them to templates. It means no escaping function in Twig templates. 2 reasons: * it reduces risks of security issue for future user made templates * more readable templates * sanitize user configuration fields after loading them.
Diffstat (limited to 'tpl/page.header.html')
-rw-r--r--tpl/page.header.html2
1 files changed, 1 insertions, 1 deletions
diff --git a/tpl/page.header.html b/tpl/page.header.html
index 0fd65e40..2d186aa2 100644
--- a/tpl/page.header.html
+++ b/tpl/page.header.html
@@ -8,7 +8,7 @@
8<div id="menu"> 8<div id="menu">
9 <ul> 9 <ul>
10 <li><span id="shaarli_title"> 10 <li><span id="shaarli_title">
11 <a href="{$titleLink}">{$shaarlititle|htmlspecialchars}</a> 11 <a href="{$titleLink}">{$shaarlititle}</a>
12 </span> 12 </span>
13 </li> 13 </li>
14 14