blob: 48c1689971cc6f1e8c6c7b3a58ad5679dedef8b3 (
plain)
1
2
3
4
5
6
7
8
9
10
11
12
13
14
15
16
17
18
19
20
21
22
23
24
25
26
27
28
29
30
31
32
33
34
35
36
|
The Atom and RSS templates use `ESCAPE=HTML` in the title elements. However, HTML-escaped characters aren't valid according to <http://feedvalidator.org/>.
Removing `ESCAPE=HTML` works fine, but I haven't checked to see if there are any characters it won't work for.
For Atom, at least, I believe adding `type="xhtml"` to the title element will work. I don't think there's an equivalent for RSS.
> Removing the ESCAPE=HTML will not work, feed validator hates that just as
> much. It wants rss feeds to use a specific style of escaping that happens
> to work in some large percentage of all rss consumers. (Most of which are
> broken).
> <http://www.rssboard.org/rss-profile#data-types-characterdata>
> There's also no actual spec about how this should work.
>
> This will be a total beast to fix. The current design is very clean in
> that all (well, nearly all) xml/html escaping is pushed back to the
> templates. This allows plugins to substitute fields in the templates
> without worrying about getting escaping right in the plugins -- and a
> plugin doesn't even know what kind of template is being filled out when
> it changes a field's value, so it can't do different types of escaping
> for different templates.
>
> The only reasonable approach seems to be extending HTML::Template with an
> ESCAPE=RSS and using that. Unfortunately its design does not allow doing
> so without hacking its code in several places. I've contacted its author
> to see if he'd accept such a patch.
>
> (A secondary bug is that using meta title currently results in unnecessry
> escaping of the title value before it reaches the template. This makes
> the escaping issues show up much more than they need to, since lots more
> characters are currently being double-escaped in the rss.)
>
> --[[Joey]]
> Update: Ok, I've fixed this for titles, as a special case, but the
> underlying problem remains for other fields in rss feeds (such as
> author), so I'm leaving this bug report open. --[[Joey]]
|