search

concerto / concerto-simple-rss

Simple RSS Dynamic Content for Concerto 2
Other
6 stars 2 forks source link

provide for sanitization of content #32

Closed mfrederickson closed 10 years ago

mfrederickson commented 10 years ago

I think it would be beneficial to be able to specify that you want script and/or img tags removed from the content so the frontend screens are not making extraneous calls.

When you watch the frontend screen network traffic when a CNN news feed is being rendered there is traffic back to CNN. If you look at the Wired feed you will sometimes see banner ads at the bottom. When RSS is placed into the Ticker feed for a scrolling marquee, the CSS hides the images, but they are still fetched resulting in wasted bandwidth.

Even though this could be handled in an XSL transformation, it would be easier for the end user if we implemented some sanitization (user specified) that ran on the content before storing it. (I'm thinking of the sanitization done on the Text content type.)

What does anyone else think about this?

bamnet commented 10 years ago

+1.

Including images by default seems safe, but I'm not sure we get any value from a Githubissues.

  • Githubissues is a development platform for aggregating issues.