将最后一个错误传递给自定义错误重定向的最佳方法？

Question

想知道您对此解决方案的看法，这是否是将错误消息传递到自定义页面的正确方法？

在 web.config 中：

    <customErrors mode="On" defaultRedirect="~/Error.aspx"></customErrors>

在 Global.asax 中：

<script RunAt="server">
    void Application_Error(object sender, EventArgs e)
    {
    Exception ex = Server.GetLastError();
    if (ex != null && Session != null)
    {
        ex.Data.Add("ErrorTime", DateTime.Now);
        ex.Data.Add("ErrorSession", Session.SessionID);
        HttpContext.Current.Cache["LastError"] = ex;
    }
    }

</script>

在我的 Error.aspx.cs 中：

protected void Page_Load(object sender, EventArgs e)
{
    if (IsPostBack) return;

    if (HttpContext.Current.Cache["LastError"] != null)
    {
        Exception ex = (Exception)HttpContext.Current.Cache["LastError"];
        if (ex.Data["ErrorTime"] != null && ex.Data["ErrorSession"] != null)
            if ((DateTime)ex.Data["ErrorTime"] > DateTime.Now.AddSeconds(-30d) && ex.Data["ErrorSession"].ToString() == Session.SessionID)
                Label1.Text = ex.InnerException.Message;
    }
}

问题：我不想从 Global.asax 进行 Server.Transfer，因为..我不知道。 对我来说似乎很笨拙。 希望能够将 customErrors 更改为 RemoteOnly。 因此必须将最后一个异常保存在某个地方，但不能是会话，因此保存到缓存中，但要保存一些额外的数据（时间和会话ID），因为缓存是全局的，并且希望确保不会向某人显示错误的错误。

---

我稍微改变了我的代码。 现在只是：

void Application_Error(object sender, EventArgs e)
{
    HttpContext.Current.Cache["LastError"] = Server.GetLastError().GetBaseException();
    Server.ClearError();
}

...并且...

protected void Page_Load(object sender, EventArgs e)
{
    if (IsPostBack) return;

    if (HttpContext.Current.Cache["LastError"] != null)
    {
        Exception ex = (Exception)HttpContext.Current.Cache["LastError"];
        if (ex != null)
            Label1.Text = ex.Message;
    }
}

注意 SessionID 如果是匿名用户则不存在，并且 ex.Data.Add 已经存在的密钥将导致错误，让我意识到调用 ClearError 很重要

Answer 1

我认为这是一个不错的方法。 这不是我这样做的方式，但我的代码太长，无法发布（并且在 VB.NET 中）。

我要更改的一件事是错误页面本身。 不要显示错误，而是考虑将文本框添加到错误页面作为可选字段，用户可以在其中输入其电子邮件地址并单击按钮将错误报告发送给您。 然后，当您收到错误报告时，您可以查看问题并回复它们。 这是一种更加用户友好的方式，并且对于我已经完成此操作的网站来说效果非常好。

按照这些思路，您可能还希望收集表单数据、会话数据和任何其他有价值的数据，并将其放入错误报告中。 这可以使诊断问题变得更加容易。

Answer 2

我们做的事情可能适合你，也可能不适合你。 我们在数据库中进行了大量的日志记录。 当我们收到错误时，我们会将其记录下来并生成错误 ID。 我们重定向到带有错误 ID 的通用页面并在那里获取详细信息。

当然，当错误是“无法连接到数据库”时，这在表面上是平淡的，但这种情况不会经常发生；）

Answer 3

由于缓存是全局的，因此不建议这样做，正如您所说，您可能会向某人显示错误的错误。 我还应该说，出于安全原因，您不应该将错误消息直接输出给最终用户。

看一下这个问题：

ASP.NET 自定义错误页面服务器GetLastError 为 null

总结如下：

Server.Transfer(String.Concat("~/Error.aspx?message=", HttpUtility.UrlEncode(ex.InnerException.Message)))

而不是依赖 ASP.NET 使用 CustomErrors 部分中的设置进行重定向。

Answer 4

我必须同意 n8wrl 和 Steve 的观点，即更好的方法是将错误记录在数据库中，然后仅向用户返回错误 ID。 他们确实不需要查看技术细节，这可能会暴露敏感信息。

在我们的例子中，我们还传入用户的 ID（如果可用）和发生错误的页面（当您到达全局 Application_Error 时，Request.URL 仍然有效）。 这样，我们就可以更容易地追踪错误。 另请注意，您不必将 Global.asax 与脚本标记一起使用。 如果您在 App_Code 目录中创建 Global.asax.cs 文件，则可以直接编写 C# 代码（但这可能取决于项目类型）。

Answer 5

Server.ClearError();

我认为这一行应该在显示 ErrorMessage 之后放置在 Error.aspx.cs 上。

Answer 6

我负责创建自定义错误页面。 一切都很简单：
在 web.config 文件中，我有：

<customErrors mode="On">
<error statusCode="404" redirect="~/error-pages/page-not-found.aspx?error=1"
</customErrors>

在 Global.asax 中，在 Application_Error 方法中：
一些代码...

Server.Transfer("~/error-pages/error.aspx");

在自定义错误页面“error.aspx”中： Server.ClearError();

我不知道到底修改了什么，但这不再有效。 当代码到达 Server.Transfer 方法时，总是会引发异常： Error waiting child request for the page...

我查看了一些解决方案，最后找到了这个。
我修改了我的代码，现在它似乎可以工作：

<customErrors mode="On" defaultRedirect="~/error-pages/error.aspx">
  <error statusCode="404" redirect="~/error-pages/page-not-found.aspx?error=1" />
</customErrors>

并且在 global.asax 方法中：

Session["LastError"] = Server.GetLastError();

它也可以使用 Cache[""] 代码，但我更喜欢 Session 变量。

所以，感谢您的回复。

• 不要忘记清除自定义错误页面中的错误。 这个很重要。 此外，还会指示不向用户准确显示所有错误。 也许采用某种用户友好的格式。 并在日志文件中提供所有信息，或通过电子邮件或其他方式发送。

希望这有用。

Answer 7

GetLastError();

请记住“此方法仅在 .asp 文件将任何内容发送到客户端之前可用。” - 来自 MSDN 文档< /a>