[Mono-dev] Please approve commit change toSystem.Web.Hosting/ApplicationHost.cs

Adar Wesley adarw at mainsoft.com
Wed Jan 17 05:28:37 EST 2007

Hi Gonzalo and all,

I looked further into the need for using the URI object to validate
the physical path passed in to the ApplicationHost.CreateApplicationHost

I found out that there were tests in ApplicationHostTests that failed, 
and were marked with "NotWorking", because of the Uri.Parse.  So this
patch actually improves the situation.

I had to tweak the exceptions thrown by this method to be compatible 
with MS.

Here is a patch (already committed) that includes these tweaks and the 
tests not marked as "NotWorking".

If you have any other comments, let me know.


Adar Wesley

-----Original Message-----
From: mono-devel-list-bounces at lists.ximian.com
[mailto:mono-devel-list-bounces at lists.ximian.com] On Behalf Of Gonzalo
Paniagua Javier
Sent: Friday, January 12, 2007 8:01 AM
To: mono-devel-list at lists.ximian.com
Subject: Re: [Mono-dev] Please approve commit change

On 1/9/07, Adar Wesley <adarw at mainsoft.com> wrote:
> Hi All,
> I would like to commit a change to ApplicationHost.  The suggested
> follows:

I've seen that this patch is already in, but it's wrong. Creating the
Uri object was done so that we throw in the same cases that MS does.
At least I would add the new Uri (blah) after getting the full path.

Mono-devel-list mailing list
Mono-devel-list at lists.ximian.com
-------------- next part --------------
A non-text attachment was scrubbed...
Name: ApplicationHost.patch
Type: application/octet-stream
Size: 3147 bytes
Desc: ApplicationHost.patch
Url : http://lists.ximian.com/pipermail/mono-devel-list/attachments/20070117/9d128533/attachment.obj 

More information about the Mono-devel-list mailing list