Tcl Library Source Code

View Ticket
Login
Ticket UUID: 639036
Title: resolve doesn't work with url like ?N=D
Type: Bug Version: None
Submitter: davidw Created on: 2002-11-15 18:40:29
Subsystem: uri Assigned To: davidw
Priority: 5 Medium Severity:
Status: Closed Last Modified: 2002-11-16 03:34:22
Resolution: Accepted Closed By: davidw
    Closed on: 2002-11-15 20:34:22
Description:
If you look at a page such as

http://www.dedasys.com/freesoftware/patches/

The HTML sources provide links like ?N=D which change
the behavior of how Apache displays the file listings.
 These url's are valid.

The problem is that 'resolve' doesn't handle these well:

uri::resolve
http://dedasys.com/freesoftware/patches/?M=A ?N=D
http://dedasys.com/freesoftware/patches/?M=A?N=D

This is not how a browser resolves these url's.
User Comments: davidw added on 2002-11-16 03:34:22:
Logged In: YES 
user_id=240

Fix committed after review by Andreas.

andreas_kupries added on 2002-11-16 03:24:28:
Logged In: YES 
user_id=75003

Looks good.

davidw added on 2002-11-16 02:59:25:

File Added - 35487: latest3.diff

Logged In: YES 
user_id=240

This patch fixes the problem, and, IMO, makes the code
cleaner and more compact.  It passes all the tests,
including those added by this patch.

Attachments: