Reported by Sebastian Hoß, Jul 17, 2009

Hi there,

i just noticed that the latest update page supplies an Atom news 
feed. That is really great but it seems to have a problem with the 
git commit messages which are signed-off. Those signed-off lines 
could contain an e-mail address which is not properly escaped. This 
prevents any newsreader from parsing the feeds correctly (Firefox 
displays nothing, Thunderbird refuses to parse, Opera seems to enter 
an infinite loop, Claws-Mail won't parse either, ...). I think the 
line:

  Signed-off-by: Sebastian Hoß <mail@shoss.de>

Should be escaped like:

  Signed-off-by: Sebastian Hoß &lt;mail@shoss.de&gt;

So "<" and ">" replaced with 
"&lt;" and "&gt;". 

After noticing that i ran the feed through the feedvalidator at 
beta.feedvalidator.org and it shows some more minor problems with 
the feed:
  - The main "feed" element is missing the 
"updated" element. This should be set to the time of the 
last update of the feed itself. As InDefero is already keeping track 
of update dates this should be rather simple to do?
  - Every "entry" element is missing the 
"author" element. This should be simple as well since 
InDefero shows us who did the update already.
  - It is possible for 2 (or more) entries to have the same ID. This 
shouldn't be possible since every entry should be unique. So with 
the current behavior some newsreaders may decide only to display the 
latest entry and skip all others. This problem happens when you edit 
the same documentation page 2 consecutively.
  - IDs for downloads are not full and valid URLs (they are 
relative).


Result for my projects feed is at: 
http://beta.feedvalidator.org/check.cgi?url=http%3A%2F%2Fdenove.indef
ero.net%2Fp%2Fcore%2Ffeed%2Ftimeline%2Ftoken%2F5cZm1j%2F

And the result for the InDefero feed is at: 
http://beta.feedvalidator.org/check.cgi?url=http%3A%2F%2Fprojects.ceo
ndo.com%2Fp%2Findefero%2Ffeed%2Ftimeline%2Ftoken%2FedamqbnWY%2F