How to test for HttpException

[Arne Garvander]

I have this code in Global.asax, but it doesn't compile yet.
What am i doing wrong?
protected void Application_Error(Object sender, EventArgs e)
{
Exception objErr = Server.GetLastError().GetBaseException();
string msg = objErr.ToString();
if (objErr.GetType is HttpException)
{
// Need to bypass 404 errors
ErrNotify.LogError(msg);
}
else
{
ErrNotify.LogError(msg);
}
Server.ClearError();
}
--
Arne Garvander
(I program VB.Net for fun and C# to get paid.)

[Laurent Bugnion]

Hi,

Arne Garvander wrote:

I have this code in Global.asax, but it doesn't compile yet.
What am i doing wrong?
protected void Application_Error(Object sender, EventArgs e)
{
Exception objErr = Server.GetLastError().GetBaseException();
string msg = objErr.ToString();

I would rather use

string msg = objErr.Message;

if (objErr.GetType is HttpException)

That should be

if ( objErr is HttpException )

{
// Need to bypass 404 errors
ErrNotify.LogError(msg);
}
else
{
ErrNotify.LogError(msg);

Did you notice that you do exactly the same in both cases?

}
Server.ClearError();
}

HTH,
Laurent
--
Laurent Bugnion, GalaSoft
Software engineering: http://www.galasoft-LB.ch
PhotoAlbum: http://www.galasoft-LB.ch/pictures
Support children in Calcutta: http://www.calcutta-espoir.ch

[Dave Sexton]

Hi Laurent,

<snip>

> Exception objErr = Server.GetLastError().GetBaseException();
string msg = objErr.ToString();

I would rather use

string msg = objErr.Message;

Exception.ToString() provides details such as the StackTrace and the
StackTraces of the InnerException (recursively). Also, when deriving from
Exception you can override ToString to add more details about the exception
that the Message property doesn't provide.

So for error logging purposes, such as the OP's code, ToString is probably a
better choice.

<snip>

--
Dave Sexton

[Laurent Bugnion]

Hi Dave,

Dave Sexton wrote:

Hi Laurent,

<snip>

>> Exception objErr = Server.GetLastError().GetBaseException();
string msg = objErr.ToString();

I would rather use

string msg = objErr.Message;

Exception.ToString() provides details such as the StackTrace and the
StackTraces of the InnerException (recursively). Also, when deriving from
Exception you can override ToString to add more details about the exception
that the Message property doesn't provide.

So for error logging purposes, such as the OP's code, ToString is probably a
better choice.

<snip>

Yes, you're right. Don't know why, I was sure that the OP used msg to
display a warning to the user.

Greetings,
Laurent
--
Laurent Bugnion, GalaSoft
Software engineering: http://www.galasoft-LB.ch
PhotoAlbum: http://www.galasoft-LB.ch/pictures
Support children in Calcutta: http://www.calcutta-espoir.ch