mwrock / RequestReduce

Instantly makes your .net website faster by reducing the number and size of requests with almost no effort.
www.requestreduce.org
Apache License 2.0
228 stars 49 forks source link

RequestReduce crash with Microsoft.AspNet.SignalR #235

Open GitOffice opened 11 years ago

GitOffice commented 11 years ago

After I install RequestReduce and Microsoft.AspNet.SignalR via Nuget! There is crash on Owin.dll and error is can not load optipng.exe , Please help , Thank you

jokokko commented 11 years ago

I experienced the same issue with SignalR 1.0.0-rc1, but rc2 fixed the problem with scanning bin folder & unamanged assemblies. Meanwhile, I had to add page filter for SignalR routes due to RR crashing the app pool:

    Stack trace:    at RequestReduce.Module.ResponseFilter.DoTransform(Byte[] buffer, Int32& startTransformPosition)
   at RequestReduce.Module.ResponseFilter.HandleMatchingStopMatch(Int32 i, Byte b, Byte[] buffer, Int32& endTransformPosition, Int32& startTransformPosition)
   at RequestReduce.Module.ResponseFilter.Write(Byte[] buffer, Int32 offset, Int32 count)
   at System.Web.HttpWriter.FilterIntegrated(Boolean finalFiltering, IIS7WorkerRequest wr)
   at System.Web.HttpResponse.UpdateNativeResponse(Boolean sendHeaders)
   at System.Web.HttpRuntime.FinishRequestNotification(IIS7WorkerRequest wr, HttpContext context, RequestNotificationStatus& status)
Request URL: http://myserver:81/signalr?transport=foreverFrame&connectionId=c209f203-6147-462b-bf44-616c9f0e752e&connectionData=[{"name":"myhub"}]&tid=10&frameId=1 

I'll take a look at this when I have the time & open a bug report if appropriate.

Edit: Meant to say RequestReduce crashing the app pool. Not SignalR. And I managed to left out the exception. I'll add it once I have access to the event logs.

GitOffice commented 11 years ago

@mwrock @jokokko , Do you fix it? If I do not install SignalR via NuGet , The RequestReduce works well. What is the meaning of 'RequestReduce crashing the app pool', can you tell me ,thx

jokokko commented 11 years ago

@GitOffice: Your problem is with SignalR (bootstrapping Katana throws when reflection contexts of assemblies in bin folder are loaded and an unmanaged assembly, e.g. optpng, is encountered), not RR. And it's a bug that was fixed in RC2.

GitOffice commented 11 years ago

@jokoko Thanks for your reply