Ticket #486 (closed defect: fixed)

Opened 1 year ago

Last modified 5 months ago

please use absolute URL in links in feed

Reported by: russ Assigned to: ebrown
Priority: high Milestone:
Component: ambra Version: 0.7
Keywords: atom, feed Cc:
Blocking: Blocked By:

Description

the combination of relative link + xml:base attribute does not play nicely with feedburner's URL rewrites (which we need for tracking)

we can't use feedburner until this is fixed.

please don't push back on me again about this. where features of the ATOM spec are unsupported by the online tools we want to use/provide to users we have to change the feed.

Dependency Graph

Change History

07/13/07 10:09:36 changed by jsuttor

  • keywords set to atom, feed.
  • status changed from new to assigned.
  • version changed from 0.7 to 0.8.
  • milestone changed from Bugs to 0.8.

07/13/07 10:17:42 changed by amit

  • version changed from 0.8 to 0.7.

This is different, in the sense that I believe we can achieve the same in a different legal way. The option would be to insert absolute URLs and take away the relative link + xml:base. Jeff or someone else will know better.

07/13/07 10:23:46 changed by russ

see also:

http://feedvalidator.org/check.cgi?url=http%3A%2F%2Fwww.plosone.org%2Ffeed%2FPLoSONE.xml

07/14/07 12:14:27 changed by amit

  • owner changed from jsuttor to ebrown.
  • status changed from assigned to new.

Eric, can you take a look at this? Thanks.

07/16/07 11:17:06 changed by russ

i also filed a support ticket with feedburner to respect the xml:base attribute. if for some reason they move fast on this i'll let you know...

07/16/07 21:24:50 changed by ebrown

  • status changed from new to closed.
  • resolution set to fixed.

(In [3207]) fixes #486 Use absolute URLs in feed links

Relative links are still avaiable if you set the cgi parameter relativeLinks to true (the default is false). For example:

http://.../article/feed?relativeLinks=true

See also: wiki:PLoSRSS?

(follow-up: ↓ 8 ) 07/17/07 11:06:04 changed by ebrown

(In [3211]) re #486 Allow manual setting of self-link and make id unique via URN

Plos will be manually pulling these feeds and making them available via apache. Thus, the need to manually set the self-link. To do so, simply add the selfLink cgi-parameter. For example:

Also, feedburner insists on a valid globally-unique feed id. So using the URN I believe we've registered, simply set the ID to

info:doi/10.1371/feed.pone

This can be set via pub.feed.id in plosone.xml. And it should probably be changed for different journals as the feeds will be different.

An initial attempt to keep this id unique for different categories and authors is done by adding cgi-parameters to the URN, but this should be revisited as feed support is expanded. I could find very little on the net indicating when doing this kind of thing is appropriate. My suggestion is to view the id as a cache key and make sure it changes if anything other than new elements end up in the feed.

(in reply to: ↑ 7 ) 07/23/07 09:41:01 changed by russ

Replying to ebrown:

Plos will be manually pulling these feeds and making them available via apache.

actually, we don't plan to continue making these feeds available in a canned format via apache if we don't have to - we'd like them to by dynamic (see #469)

Also, feedburner insists on a valid globally-unique feed id. So using the URN I believe we've registered, simply set the ID to info:doi/10.1371/feed.pone

hmm...feedburner seems to be working with the current feeds even though id is blank...

07/16/08 11:00:34 changed by

  • milestone deleted.

Milestone 0.8 deleted