ServiceStack catch(WebServiceException ex) - 有错误的ErrorCode

时间:2013-02-20 14:38:53

标签: exception-handling servicestack

在我的ServiceStack服务中,我抛出一个有内部异常的异常。当我在客户端捕获WebServiceRequest时,ErrorCode是内部异常类型名称。

这对我不好,因为它不允许我回应服务器上抛出的特定异常类型。

我无法理解为什么ServiceStack是这样设计的。通常会捕获较低级别的异常并将其包含在内容更丰富且有时最终用户友好的异常中。

如何更改默认行为,使其使用表面级异常,而不是最内层?

2 个答案:

答案 0 :(得分:3)

https://github.com/ServiceStack/ServiceStack/wiki/Error-Handling查看第一个示例后,我决定在DtoUtils处查看.HandleException,如下所示:

    public static object HandleException(IResolver iocResolver, object request, Exception ex)
    {
        if (ex.InnerException != null && !(ex is IHttpError))
            ex = ex.InnerException;

        var responseStatus = ex.ToResponseStatus();

        if (EndpointHost.DebugMode)
        {
            // View stack trace in tests and on the client
            responseStatus.StackTrace = GetRequestErrorBody(request) + ex;
        }

        Log.Error("ServiceBase<TRequest>::Service Exception", ex);

        if (iocResolver != null)
            LogErrorInRedisIfExists(iocResolver.TryResolve<IRedisClientsManager>(), request.GetType().Name, responseStatus);

        var errorResponse = CreateErrorResponse(request, ex, responseStatus);

        return errorResponse;
    }

第一条指令用它的内部异常替换异常。我不确定那个想法是什么。这对我来说似乎很直观,所以我只是在我的AppHost类中重新实现了该方法,删除了第一个if语句块:

    public override void Configure(Container container)
    {
        ServiceExceptionHandler += (request, exception) => HandleException(this, request, exception);
    }

    /// <remarks>
    /// Verbatim implementation of DtoUtils.HandleException, without the innerexception replacement.
    /// </remarks>
    public static object HandleException(IResolver iocResolver, object request, Exception ex)
    {
        var responseStatus = ex.ToResponseStatus();

        if (EndpointHost.DebugMode)
        {
            // View stack trace in tests and on the client
            responseStatus.StackTrace = DtoUtils.GetRequestErrorBody(request) + ex;
        }

        var log = LogManager.GetLogger(typeof(DtoUtils));
        log.Error("ServiceBase<TRequest>::Service Exception", ex);

        if (iocResolver != null)
            DtoUtils.LogErrorInRedisIfExists(iocResolver.TryResolve<IRedisClientsManager>(), request.GetType().Name, responseStatus);

        var errorResponse = DtoUtils.CreateErrorResponse(request, ex, responseStatus);

        return errorResponse;
    }

这显然不太理想,因为我不得不复制一堆与原始实现问题完全无关的代码。每当我更新ServiceStack时,我都觉得我必须维护这个方法。我希望能有更好的方法来实现这一目标。

无论如何,我在客户端代码中有我喜欢的异常处理:

catch (WebServiceException ex)
{
    if (ex.ErrorCode == typeof (SomeKindOfException).Name)
    {
        // do something useful here
    }
    else throw;
}

答案 1 :(得分:1)

您似乎不必维护一堆代码。您正在编写一种方法来实现自己的错误处理。您可以尝试在自己的方法中调用DtoUtils.HandleException(this,request,exception)并修改返回的HttpError对象。不确定您是否有权更改您正在寻找的所有属性/值。

public static object HandleException(IResolver iocResolver, object request, Exception ex)
{
    HttpError err = (HttpError)DtoUtils.HandleException(this, request, ex);
    err.Reponse = ex.InnerException; 
}