Forum .LRN Q&A: news-aggregator portlet error

Collapse
Posted by Eduardo Palacio on
Hi all:

in the news-aggregator-potlet the news don´t show right, because the variable @items.content@ show all html the tags.

This happens in versions 2.0.0rc1, 2.0.1 and 2.0.2

you need change this variable  @items.content@ in news-aggregator-portlet.adp for this @items.content;noquote@
an all works well.

I try this url for test http://www.libertaddigital.com/rss/portada.xml

bye.

Collapse
Posted by Nima Mazloumi on
Hi Eduardo,
there were several posts on that. I will write a bug report and submit a patch today and if it is approved it will be fixed on cvs.

Greetings,
Nima

Collapse
Posted by Felipe Gelbcke Gubert on
Same with weblog... (if you post a link)

@entries-content@

@entries-content;noquote@

solves the problem

Collapse
Posted by Jeff Davis on
You need to be careful making things like that noquote since if someone were to put something like the following: in
<img src="http://yoursite.com/admin/grant-admin?user_id=eviluser">
in an aggregated feed you read on yoursite.com. When you went to read it, it would get the url which would grant sitewide admin to thier user_id on your site.

In general, anything that comes from the outside needs to be checked for XSS and allowed tags before it is displayed.