Need Better Approach For Specifying RSS/Atom Images

Description

Currently you need to create a distinct rss/atom because the image is hard coded. Because we'll have distinct feeds per category, series, etc. there will be a ridiculous amount of feed configs.

Activity

Show:
A
August 31, 2011, 10:14 PM
Edited

Josh Holtzman added a comment - 22/Aug/11

I would argue that allowing the query string to determine cover art is dangerous, since it allows anyone to manipulate the feed and associate arbitrary images to our feeds.

Is this feature request simply a workaround. Are you looking for an inexpensive way to customize feeds on the fly?

Sergio Feria added a comment - 22/Aug/11 6:05 AM
Instead of overriding the cover art via a querystring parameter in the URL, I think what would solve our needs is to define the image in terms of another property. For example, to be able to define the image as being derived from the category,

feed.cover=/engage/[category]_cover.png

This would enable us to customize feeds easily without an individual feed config for each category.

Adam Hochman added a comment - 27/Aug/11 1:56 AM
The former is a bug. The latter is a feature request. Locally we need a way to specify cover art per series, and currently the only way to do this is by creating a unique feed per subject. Sergio's suggestion seems to be a more succinct approach, but I am open to other cheap and easy (and secure) approaches. Thoughts?

Josh Holtzman
September 4, 2011, 6:01 PM

Which category, Sergio? Series created in the admin UI have titles, organizers, contributors, subjects, languages, licenses, descriptions, and privileges.

Would an image-per-series suffice? If so, the SeriesFeedService could grab the URL for the series, and display it as part of the feed.

Sergio Feria
September 9, 2011, 11:46 PM

Could a property like feed.cover_source in conf/feeds/feed.properties, determine if the feed's cover image was static and defined by feed.cover or more dynamically by category or some other means?

What I would like to see is the ability to have an extension of AbstractFeedService add the itunes:image, determined by the series' subject id or name.
For example if the subject of "Engineering, Computer Science" is chosen, and the itunes:category would be '101102', then an image of '101102.jpg' be included or 'comp_science.jpg' if we want to map codes to names.

[Thinking further, a series should have the ability to override the default image with a custom one uploaded to the series edit page, but that's not in the scope of this.]

Josh Holtzman
September 11, 2011, 8:48 PM

Please see comments in MH-4180.

The approach described there is to set "cover.art.url.template" (currently set to http://webcast.berkeley.edu/images/itcategories/{0}.png in the UCB 1.2.x branch) to a URL containing the category images.

As long as the series have the appropriate itunes categories set (which they don't in the trunk, as far as I could tell), the feeds will include proper images.

Assignee

Sergio Feria

Reporter

A

Tags (folksonomy)

None

Components

Fix versions

Affects versions

Priority

Major
Configure