Roundup Tracker - Issues

Message1142

Author micktwomey
Recipients
Date 2004-03-24.10:49:49
Message-id
In-reply-to
Logged In: YES 
user_id=1004574

Apologies, if I gave the impression that I filed the bug originally, I simply 
encountered the same problem and piggy backed my comments on the 
bug report :)

To answer your questions:

* I'm using Thunderbird on Mac OS X, however co-workers using either 
mozilla or evolution are encountering the same problem (in fact it's how 
I found out about it, I added myself via the admin account, they 
registered).

* Looking at the URL more closely it is indeed "damaged", in so far as 
Mozilla picks up every character right up to the last one as a URL, but 
omits he last character. I think it might be it's URL selection logic at 
fault. Adding the missing character makes it work.

Changing the email template to add an extra newline after the url fixes 
the problem:

$ diff -u roundup/cgi/client.py.original roundup/cgi/client.py
--- roundup/cgi/client.py.original      2004-03-24 10:33:45.000000000 
+0000
+++ roundup/cgi/client.py       2004-03-24 10:33:53.000000000 +0000
@@ -794,6 +794,7 @@
 please visit the following URL:
  
    %(url)s?@action=confrego&otk=%(otk)s
+
 '''%{'name': props['username'], 'tracker': tracker_name, 'url': self.base,
                 'otk': otk}
         if not self.sendEmail(props['address'], subject, body):

(the diff might be slightly mangled, but you get the idea).

So this is actually a bug in Mozilla :)

* As for the unit test error, it's not on an NFS disk, I'll see if I can pin it 
down more, it might be a bug in one of the libraries involved, some of 
them are a little old on the Java Desktop :/

* I'll follow up on the RFE, I've joined the list.
History
Date User Action Args
2009-02-03 14:20:37adminlinkissue906247 messages
2009-02-03 14:20:37admincreate