相关问题:
Web API ApiController PUT and POST methods receive null parameters intermittently
背景
在对现有Web API项目进行负载测试时,由于参数在发布到操作时为null,因此我注意到很多空引用异常.
原因似乎是在dev环境中运行时为日志请求注册的自定义消息处理程序.删除此处理程序可解决此问题.
我知道在Web API中我只能读取一次请求体,并且读取它总是会导致我的参数为null,因为模型绑定将无法进行.出于这个原因,我正在使用带有ContinueWith的ReadAsStringAsync()方法来读取正文.看起来这在~0.2%的请求中表现奇怪(在使用Apache Bench的本地调试期间).
码
在最基本的层面上,我有以下几点:
模型
public class User
{
public string Name { get; set; }
}
API控制器
public class UsersController : ApiController
{
[HttpPost]
public void Foo(User user)
{
if (user == null)
{
throw new NullReferenceException();
}
}
}
消息处理程序
public class TestMessageHandler : DelegatingHandler
{
protected override Task<HttpResponseMessage> SendAsync(HttpRequestMessage request, CancellationToken cancellationToken)
{
request.Content.ReadAsStringAsync().ContinueWith((task) =>
{
/* do stuff with task.Result */
});
return base.SendAsync(request, cancellationToken);
}
}
…在应用程序启动期间注册的…
GlobalConfiguration.Configuration.MessageHandlers.Add(new TestMessageHandler());
我正在使用WebAPI 4.0.30506.0,这是发布时的最新版本.项目中的所有其他MS软件包也运行最新版本(下面链接的演示项目现已更新,以反映这一点).
测试
使用Loadster运行初始测试是针对带有.NET 4.0.30319的Server 2008 R2上的负载平衡IIS 7.5设置运行的.我正在使用Apache Bench在.NET 7上使用.NET 4.5.50709在IIS 7.5上本地复制它.
ab -n 500 -c 25 -p testdata.post -T "application/json" http://localhost/ModelBindingFail/api/users/foo
其中testdata.post包含
{ "Name":"James" }
通过这个测试,我发现500个请求大约有1个失败,所以~0.2%.
下一步…
如果您想尝试自己,我已经将我的演示项目放在GitHub上,尽管除了我上面发布的它是一个标准的空Web API项目.
也很乐意尝试任何建议或发布更多信息.谢谢!
最佳答案 我仍然在调查这个的根本原因但到目前为止,我的直觉是,ContinueWith()正在不同的上下文中执行,或者在请求流被处理的那个点或类似的东西上执行(一旦我认为我肯定会更新这一段).
在修复方面,我已经快速测试了三个可以处理500个请求而没有错误.
最简单的是只使用task.Result,但它有一些问题(它可以apparently cause deadlocks,虽然YMMV).
protected override Task<HttpResponseMessage> SendAsync(HttpRequestMessage request, CancellationToken cancellationToken)
{
var result = request.Content.ReadAsStringAsync().Result;
return base.SendAsync(request, cancellationToken);
}
接下来,你可以确保你正确地链接你的延续,以避免任何关于上下文的歧义,但它是非常难看的(我不是100%确定它是否是副作用免费):
protected override Task<HttpResponseMessage> SendAsync(HttpRequestMessage request, CancellationToken cancellationToken)
{
var result = request.Content.ReadAsStringAsync().ContinueWith(task =>
{
/* do stuff with task.Result */
});
return result.ContinueWith(t => base.SendAsync(request, cancellationToken)).Unwrap();
}
最后,最佳解决方案似乎使用async / await到sweep away any threading nasties,显然如果你遇到.NET 4.0,这可能是一个问题.
protected override async Task<HttpResponseMessage> SendAsync(HttpRequestMessage request, CancellationToken cancellationToken)
{
var content = await request.Content.ReadAsStringAsync();
Debug.WriteLine(content);
return await base.SendAsync(request, cancellationToken);
}