[PATCH 1/1] fix html syntax in email-gravatar.*

2014-01-14 Thread Christian Hesse
* make ampersand a html entity
* add required alt attribute
* add required img end tag
---
 filters/email-gravatar.lua | 2 +-
 filters/email-gravatar.py  | 2 +-
 2 files changed, 2 insertions(+), 2 deletions(-)

diff --git a/filters/email-gravatar.lua b/filters/email-gravatar.lua
index 9feb611..22f0641 100644
--- a/filters/email-gravatar.lua
+++ b/filters/email-gravatar.lua
@@ -15,7 +15,7 @@ function filter_open(email, page)
 end
 
 function filter_close()
-   html(img src='//www.gravatar.com/avatar/ .. md5 .. ?s=13d=retro' 
style='height:10pt;width:10pt'  .. buffer)
+   html(img src='//www.gravatar.com/avatar/ .. md5 .. 
?s=13amp;d=retro' style='height:10pt;width:10pt' alt='Gravatar' /  .. 
buffer)
 end
 
 function filter_write(str)
diff --git a/filters/email-gravatar.py b/filters/email-gravatar.py
index 5d08ea8..97f21cf 100755
--- a/filters/email-gravatar.py
+++ b/filters/email-gravatar.py
@@ -36,4 +36,4 @@ sys.stdout = codecs.getwriter(utf-8)(sys.stdout.detach())
 md5 = hashlib.md5(email.encode()).hexdigest()
 text = sys.stdin.read().strip()
 
-print(img src='//www.gravatar.com/avatar/ + md5 + ?s=13d=retro' 
style='height:10pt;width:10pt'  + text)
+print(img src='//www.gravatar.com/avatar/ + md5 + ?s=13amp;d=retro' 
style='height:10pt;width:10pt' alt='Gravatar' /  + text)
-- 
1.8.5.2

___
CGit mailing list
CGit@lists.zx2c4.com
http://lists.zx2c4.com/mailman/listinfo/cgit


Re: [PATCH 1/1] fix html syntax in email-gravatar.*

2014-01-14 Thread Jason A. Donenfeld
Good catch with amp;. Merged.
___
CGit mailing list
CGit@lists.zx2c4.com
http://lists.zx2c4.com/mailman/listinfo/cgit