Edgewall Software
Modify

Opened 15 years ago

Closed 15 years ago

#601 closed defect (fixed)

Links in RSS feed are missing schemes, wiki links

Reported by: trac@… Owned by: daniel
Priority: high Milestone: 0.8
Component: general Version: devel
Severity: major Keywords:
Cc: Branch:
Release Notes:
API Changes:

Description

      <image>
        <title>sabi.net</title>
        <url>svn.sabi.net/trac/common/sabi-small.jpg</url>
        <link>svn.sabi.net/trac/dev/timeline</link>
      </image>
        <link>svn.sabi.net/trac/dev/wiki/RecentChanges</link>
        <description>Wiki page <a href="svn.sabi.net">RecentChanges</a>
                     edited by nicholas</description>

and so forth.

Attachments (0)

Change History (8)

comment:1 by anonymous, 15 years ago

In [774] usage of a non-standard env variable, SCRIPT_URI, was added. It only exists if you're using mod_rewrite with apache.

comment:2 by daniel, 15 years ago

Milestone: 0.8
Priority: normalhigh

comment:3 by daniel, 15 years ago

Resolution: fixed
Status: newclosed

Fixed in [819].

comment:4 by dju`, 15 years ago

Resolution: fixed
Status: closedreopened

rss links are missing schemes on this website (http://projects.edgewall.com/trac/timeline?daysback=90&max=50&format=rss)

<a href="://projects.edgewall.com">TracGuide</a>

so i'm reopening it, at least to make the website rss links usable.

comment:5 by xian (at) mintchaos . com, 15 years ago

Keywords: rss added

The timeline rss feeds are the only ones that suffer from this. It seems to be a problem with $HTTP.Protocol that is being fed to the hdf.

Here is how it looks in source:file/trunk/templates/timeline_rss.cs#825

  <?cs set base_url = $HTTP.Protocol+'://'+$HTTP.Host ?>

I fixed it in my local copy by changing my template to match how the URL is set in the other rss templates such as: source:trunk/templates/log_rss.cs#825

   <?cs set base_url = "http://"+$HTTP.Host ?>

This of course breaks https:// functionality. But considering that the other feeds do it, and it didn't work at all before, I don't mind.

comment:6 by trac@…, 15 years ago

Well, I run trac behind https, and I'm the reporter, so… :-)

comment:7 by daniel, 15 years ago

Keywords: rss removed
Status: reopenednew

comment:8 by daniel, 15 years ago

Resolution: fixed
Status: newclosed

Should be fixed in [831].

Modify Ticket

Change Properties
Set your email in Preferences
Action
as closed The owner will remain daniel.
The resolution will be deleted. Next status will be 'reopened'.
to as closed The owner will be changed from daniel to the specified user.

Add Comment


E-mail address and name can be saved in the Preferences .
 
Note: See TracTickets for help on using tickets.