Edgewall Software

Opened 18 years ago

Closed 18 years ago

Last modified 16 years ago

#259 closed defect (fixed)

trac.cgi breaks if called from a prefix-less URL (trivial fix included)

Reported by: pjd@… Owned by: Jonas Borgström
Priority: normal Milestone: 0.7
Component: general Version: 0.6.1
Severity: normal Keywords:
Cc: Branch:
Release Notes:
API Changes:
Internal Changes:


With an Apache VirtualHost entry looking something like this:

<VirtualHost *>
	ServerName trac.example.org
	ScriptAlias / /usr/local/share/trac/cgi-bin/trac.cgi/
	<Location />
		SetEnv TRAC_DB /home/trac/example.db

…trac dies with an IndexError in href_join() in source:trac/util.py.


href_join() assumes all its arguments have a non-zero length.


--- trac/util.py.orig   Mon Mar 29 13:15:48 2004
+++ trac/util.py        Fri Apr 16 16:52:21 2004
@@ -86,11 +86,7 @@

 def href_join(u1, *tail):
     for u2 in tail:
-        if u1[-1] == '/' and u2[0] != '/' or \
-            u1[-1] != '/' and u2[0] == '/':
-                u1 = u1 + u2
-        else:
-            u1 = u1 + '/' + u2
+        u1 = u1.rstrip('/') + '/' + u2.lstrip('/')
     return u1

 def add_dictlist_to_hdf(list, hdf, prefix):

Attachments (0)

Change History (4)

comment:1 by Jonas Borgström, 18 years ago

Resolution: fixed
Status: newclosed

Ok, you're right. I've applied your patch in [419].


comment:2 by pjd@…, 18 years ago

Will this get included in the 0.6.2 release?

comment:3 by daniel, 18 years ago


comment:4 by anonymous, 16 years ago

hi there

Modify Ticket

Change Properties
Set your email in Preferences
as closed The owner will remain Jonas Borgström.
The resolution will be deleted. Next status will be 'reopened'.
to The owner will be changed from Jonas Borgström 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.