Edgewall Software

Ticket #2290 (closed defect: fixed)

Opened 3 years ago

Last modified 2 years ago

Invalid RSS feed for timeline

Reported by: mrenzmann@… Owned by: cmlenz
Priority: normal Milestone: 0.9.3
Component: timeline Version: 0.9b2
Severity: normal Keywords: rss timeline
Cc:

Description

I noticed that the following ticket broke RSS feed validation: http://madwifi.org/ticket/96

Feedvalidator.org reported the following problems:

line 176, column 18: Undefined title element: em

    <title>Ticket <em title="ath/if_ath.c: Incorrect use of dev-&gt;name
                  ^
line 177, column 118: XML parsing error: <unknown>:177:118: mismatched tag

... Moraes Holschuh &lt;hmh@debian.org&gt;</title>
                                             ^

The offending item:

   <item>
    <title>Ticket <em title="ath/if_ath.c: Incorrect use of dev-&gt;name

hal; regdomain likely 96 country code 62">#96 (defect) created by Henrique de Moraes Holschuh &lt;hmh@debian.org&gt;</title>
     <author>Henrique de Moraes Holschuh &lt;hmh@debian.org&gt;</author>
    <pubDate>Mon, 31 Oct 2005 22:00:05 GMT</pubDate>
    <link>http://madwifi.org/ticket/96</link>
    <description>ath/if_ath.c: Incorrect use of dev-&amp;gt;name

hal; regdomain likely 96 country code 62</description>
   </item>

First guess from IRC:

<cmlenz> otaku42: I think that's an error in how the timeline RSS feed removes HTML tags :-/

Attachments

timeline-rss.txt (11.1 KB) - added by mrenzmann@… 3 years ago.
plain text version of the RSS feed that was tested by feedvalidator.org

Change History

Changed 3 years ago by mrenzmann@…

plain text version of the RSS feed that was tested by feedvalidator.org

Changed 3 years ago by cmlenz

The ticket summary causes this is

“ath/if_ath.c: Incorrect use of dev->name hal; regdomain likely 96 country code 62”.

I think the / and > characters are confusing the regexp that removes HTML tags for the RSS feed.

Changed 3 years ago by cmlenz

  • milestone changed from 0.9.1 to 0.9.2

Changed 3 years ago by mgood

Actually, it's the newline in the ticket summary that's the offender here. Were these tickets imported from another source? Normally the summaries wouldn't have any newlines since they're entered in a single line input box.

Changed 3 years ago by cmlenz

  • owner changed from jonas to cmlenz
  • status changed from new to assigned

This should be fixed in trunk since [2685]. I'll mark this ticket as closed once we've merged those changes intro 0.9-stable.

Changed 3 years ago by cmlenz

  • status changed from assigned to closed
  • resolution set to fixed

Fixed in 0.9-stable in [2724].

Add/Change #2290 (Invalid RSS feed for timeline )

Author



Change Properties
<Author field>
Action
as closed
Next status will be 'reopened'
to The owner will change from cmlenz. Next status will be 'closed'
 
Note: See TracTickets for help on using tickets.