Timeline RSS feed produces broken links

[tjb@…]

When trying to view the links provided by the timeline RSS feed, the fully qualified hostname is being truncated.

Our URLS normally look like

    http://fred.cea.com.au/trac/ProjectName/...

but the links provided by the RSS feed are coming out as

    http://fred.cea.com/trac/ProjectName/...

This has broken sometime in the last few weeks, not sure when.
We are running from the latest trunk available.

[Christian Boos]

Check you trac.ini file, you most probably have this setting:

[trac]
base_url = http://fred.cea.com

which should be changed to:

[trac]
base_url = http://fred.cea.com.au

[tjb@…]

No, my base_url looks like this

[trac]
  base_url = http://fred.cea.com.au/trac/

If I remove the trailing / (giving base_url = ​http://fred.cea.com.au/trac), I get links from the RSS feed which look like ​http://fred.cea.co/trac/ProjectName/…

If it works for you, do you have a 4 part url? Looking into the code, the timeline rss feed is produced by a genshi template. My Genshi is 0.3.4, I'll look for an update…

[tjb@…]

I updated to Genshi 0.3.5, and the current trac head - r4391.

Still the same problem. Looking at the Genshi template, there is a for loop (for event in events), and a method called on event (event.abs_href(req)), which causes my problem. Earlier use of abs_href (eg, abs_href.timeline()) produce the correct output, but of course, it is a different method.

I don't have the time to devote to learning Genshi and finding the problem, so I am still stuck :-(

[Christian Boos]

Sorry, yes this appears to be a real issue.

[Christian Boos]

Thomas Moschny reminded us about this issue; the sooner it gets fixed, the better :)

[Matthew Good]

The abs_href truncation seems to be incorrect: req.abs_href.base[:-len(req.href.base)]

I'll work on a fix.

[Matthew Good]

Ah, there actually is a config problem here, though I may make a change to the code so it doesn't rely on the req.href.base value.

The base_url setting should be the absolute URL to the base of that project. So, it should be set to:

base_url = http://fred.cea.com.au/trac/ProjectName

[thomas.moschny@…]

For my site, all urls in the generated RSS feed (and, fwiw, in the notification mails) are relative. Not sure whether this is exact the same problem, but at least strongly related.

Unlike the original submitter's setup, there are no projects, and there is no /trac/ element in the url; so all trac urls are 'top-level'.

I tried both, setting base_url to "​http://trac.domain.tld/" and to "​http://trac.domain.tld" (i.e. with and without trailing slash), but without success.

[thomas.moschny@…]

Using r4585, the problem seems (at least for me, can't speak for the original submitter) mostly solved. The values of the rss/channel/item/link elements are correct now.

What is still wrong (relative) is the value of the rss/channel/image/url element.

[tjb@…]

I tested my original problem against a recent trunk release (r4688), and the URL's now seem correct. I also upgraded to Genshi 0.3.6, as required. From my point of view, this could be closed now…

I have noted the config problem, not sure if it applies to us, but I will check.
I apologise for going so long without any input, I have been overseas on leave since late December, and only just returned.

Thanks to all involved.

[Christian Boos]

Replying to thomas.moschny@gmx.de :

Using r4585, … What is still wrong (relative) is the value of the rss/channel/image/url element.

So this seems to be the last thing to be checked.

[Matthew Good]

The main issue was fixed in r4585 and the image URL has been fixed in r4983.