如何在C#WCF REST POST中的StreamReader.ReadToEnd()之后抛出WebFaultException?

时间:2013-11-14 19:22:59

标签: c# wcf rest post stream

我有一个WCF REST项目,它将http状态代码作为WebFaultExceptions返回。这对于GET调用非常有效,但是我遇到了为POST调用返回WebFaultException的问题。请求正文中的数据使用内容类型为“application / x-www-form-urlencoded; charset = utf-8”。我认为问题是当上下文切换出using子句以抛出WebFaultException时,底层请求流被关闭。

如果我在“using”子句之前抛出WebFaultException,则会按预期返回异常。如果我从“using”子句中抛出WebFaultException,则异常不会返回给客户端。

在使用流读取器读取请求主体时,是否有人建议如何成功抛出WebFaultException?

这是我的服务器端代码的缩写版本。请注意,此示例的httpstatuscodes对于我的实际实现是不现实的。

[WebInvoke(Method = "POST"
, UriTemplate = "urls/{id}"
, BodyStyle = WebMessageBodyStyle.WrappedRequest
)]
public string PostItem(string id, object streamdata)
{

    int _id = 0;
    if (int.TryParse(companyIdSr2, out _id))
    {
        using (System.IO.StreamReader reader = new System.IO.StreamReader(streamdata))
        {
            string body = reader.ReadToEnd();

            if(string.IsNullOrEmpty(body))
            {
                // this exception doesn't make it back to the client's request object
                ThrowError(HttpStatusCode.BadRequest, "empty body");
            }   
        }
    }
    else
    {
        // this exception is successfully returned to the client's request object
        ThrowError(HttpStatusCode.BadRequest, "invalid id");        
    }    
}

private static void ThrowError(HttpStatusCode status, string message)
{
    request_error error = new request_error
    {
        request_url = WebOperationContext.Current.IncomingRequest.UriTemplateMatch.RequestUri.OriginalString,
        error_status_code = status.ToString(),
        error_message = message,
    };

    throw new WebFaultException<request_error>(error, status);
}

public class request_error
{
    [XmlElement("request_url")]
    public string request_url { get; set; }
    [XmlElement("error_status_code")]
    public string error_status_code { get; set; }
    [XmlElement("error_message")]
    public string error_message { get; set; }
}

我已经看到了这个问题 - Wrong WebFaultException when using a Stream and closing the stream - 虽然它在某种程度上解决了这个问题,但未解决的问题是不管理或关闭流是否合理。

非常感谢,

特里

1 个答案:

答案 0 :(得分:3)

您链接到的question现已收到一个解决您问题的已接受答案。

你说:

  

虽然它在某种程度上解决了这个问题,但是没有答案就是不管理或关闭流是否合理。

对此,答案说明:

  

但最好不要丢弃StreamReader,因为它无法清除任何非托管资源。

为了支持这一点,我们在另一个StackOverflow线程上得到以下答案,该线程主张调用Dispose()为何重要:https://stackoverflow.com/a/2548694/700926

要解决您在问题中提出的初始问题(WebFaultException未发送回客户端)我最终按照this answer中的建议进行操作 - 继承System.IO.StreamReader并覆盖Close所以它告诉StreamReader只重新关联非托管资源而不关闭流。

我的WcfFriendlyStreamReader看起来像这样:

public class WcfFriendlyStreamReader : StreamReader
{
    public WcfFriendlyStreamReader(Stream s) : base(s) { }

    public override void Close()
    {
        base.Dispose(false);
    }
}

MSDN documentation调用Dispose(false)看,只释放非托管资源。正如反编译器所揭示的那样,这也会导致Stream保持打开状态,这似乎可以解决问题:

protected override void Dispose(bool disposing)
{
  try
  {
    if (this.LeaveOpen || !disposing || this.stream == null)
      return;
    this.stream.Close();
  }
  finally
  {
    if (!this.LeaveOpen && this.stream != null)
    {
      this.stream = (Stream) null;
      this.encoding = (Encoding) null;
      this.decoder = (Decoder) null;
      this.byteBuffer = (byte[]) null;
      this.charBuffer = (char[]) null;
      this.charPos = 0;
      this.charLen = 0;
      base.Dispose(disposing);
    }
  }
}