search

alistapart / AListApart

The A List Apart front end repo
http://alistapart.com/
MIT License
671 stars 68 forks source link

RSS tag list (dc:subject) contains HTML #315

Closed abackstrom closed 9 years ago

abackstrom commented 9 years ago

The <dc:subject> portion of the RSS feed currently contains HTML. This breaks in NewsBlur, my RSS reader of choice. I'm not intimately familiar with RSS or the Dublin Core but I don't believe the field should contain HTML.

<dc:subject><![CDATA[<a href="/topic/css"><span itemprop="about">CSS</span></a>,
<a href="/topic/web-strategy"><span itemprop="about">Web Strategy</span></a>]]></dc:subject>
nelsonjchen commented 9 years ago

For reference, it blows up like the pictures mentioned in this problem report on Newsblur. As a user of Newsblur, I have never seen another feed do anything like this.

https://getsatisfaction.com/newsblur/topics/the_tags_for_the_posts_from_a_list_apart_are_not_parsed_correctly

aparkening commented 9 years ago

Thanks for letting us know (and providing some helpful screenshots), @abackstrom and @nelsonjchen. We'll investigate!

nelsonjchen commented 9 years ago

Whoops, sorry about that last comment which I just deleted, accidently mashed comment when closing the tab.

Thanks for looking into this!

samuelclay commented 9 years ago

Hey, I run NewsBlur and would be happy to help in any way.

aparkening commented 9 years ago

Hey @samuelclay, Michelle will be following up on this thread shortly, as she removed the HTML from the RSS feed.

You mentioned that you emailed us and didn't receive a response. Would you mind letting me know which email address you used? I don't see an email from you in our usual inboxes, so I'd like to investigate further.

samuelclay commented 9 years ago

I couldn't find the email, which means I probably never sent it. I send quite a few of these types of emails to publishers, so I assumed I did the same here. That's too bad, too, since it looks like you folks are on top of issues. Wish I would've emailed sooner! Thanks for your help, too.

michellekondou commented 9 years ago

Hi @samuelclay! as @aparkening mentioned I've cleared the html from the dc:subject. I'm still not seing the change reflected in newsblur.com - the feed probably needs to be re-fetched or updated?

nelsonjchen commented 9 years ago

I believe you can force fetch feeds from the site settings.

site settings

But I'm getting errors when I force it to refresh from A List Apart.

error

michellekondou commented 9 years ago

Thank you @nelsonjchen, I actually tried that and got the same error!

samuelclay commented 9 years ago

Looks good! NewsBlur doesn't update stories based on tag changes, so I just wiped out the existing stories and let NewsBlur take care of the rest. The stories came in just fine. Thanks!

aparkening commented 9 years ago

Thanks for reaching out @abackstrom, @nelsonjchen, and @samuelclay. Please let us know if you spot any other issues in the future!