[Mono-bugs] [Bug 47573][Wis] New - Bugs in System.Uri

[bugzilla-daemon@bugzilla.ximian.com]

Please do not reply to this email- if you want to comment on the bug, go to the
URL shown below and enter your comments there.

Changed by atsushi@ximian.com.

http://bugzilla.ximian.com/show_bug.cgi?id=47573

--- shadow/47573	2004-06-08 17:07:51.000000000 -0400
+++ shadow/47573.tmp.18831	2004-06-08 17:07:51.000000000 -0400
@@ -0,0 +1,85 @@
+Bug#: 47573
+Product: Mono: Class Libraries
+Version: unspecified
+OS: unknown
+OS Details: 
+Status: NEW   
+Resolution: 
+Severity: Unknown
+Priority: Wishlist
+Component: System
+AssignedTo: atsushi@ximian.com                            
+ReportedBy: duncan@ximian.com               
+QAContact: mono-bugs@ximian.com
+TargetMilestone: ---
+URL: 
+Cc: 
+Summary: Bugs in System.Uri
+
+Having fixed #45614, here are more bugs in our Uri implementation.
+
+I'm using the examples listed in RFC 2396, Section C.1 as the nominative
+behavior for our implementation.
+
+Currently, these 5 cases fail:
+
+With base URI = http://a/b/c/d;p?q:
+
+ 1) g:h => g:h                  (mono: http://a/b/c/g:h)
+ 2) /g  => http://a/g           (mono: file://x, /g) (!)
+ 3) //g => http://g             (mono: http://a//g)
+ 4) ?y  => http://a/b/c/?y      (mono: crash)
+ 5) #s  => (current document)#s (mono: crash)
+
+With the MS 1.1 runtime, these two cases fail:
+
+ 1) g:h (MS: UriFormatException)
+ 2) /g  (MS: UriFormatException)
+
+------- Additional Comments From duncan@ximian.com  2003-08-12 21:40 -------
+Okay, turns out case #2 isn't a bug, it was just a typo:
+
+2) /g  =>file://x, /g
+
+We actually correctly yield the result 'http://a/g'.
+
+
+------- Additional Comments From duncan@ximian.com  2003-08-13 09:07 -------
+Just found another case that's broken.
+
+ Uri u = new Uri ("mailto:test@foo.com");
+ Console.WriteLine (u);
+
+Right now we get 'file://mailto:test@foo.com/', instead we want just
+'mailto:test@foo.com'.
+
+Fixing this case should also fix the 'g:h' case. Right now, with my
+uncommited fixes, we get 'g://h/'.
+
+We need to add a stricter test to find out the host part of the URI
+and be less aggressive with appending the "//" delimiter.
+
+
+------- Additional Comments From duncan@ximian.com  2003-08-13 14:54 -------
+Created an attachment (id=5112)
+Current patch to Uri, still 2 more cases to go... if you use this class, please review
+
+
+------- Additional Comments From duncan@ximian.com  2003-08-15 17:37 -------
+Created an attachment (id=5148)
+Final patch, this should fix all the problems listed
+
+
+------- Additional Comments From duncan@ximian.com  2003-08-15 17:37 -------
+Now I need to run the testsuite, or have someone teach me how (I never
+figured out how to run only one test)...
+
+Any volunteers?
+
+------- Additional Comments From gonzalo@ximian.com  2003-08-15 21:49 -------
+make test && make run-test in mcs/class/System will give you the
+command. If you want to run a single class, add a
+/fixture:MonoTests.System.XXXXXXXXXXXx to that command line.
+
+------- Additional Comments From bmaurer@users.sf.net  2003-12-25 14:46 -------
+Duncan, what is that status on this bug?