Sporadically crashes on "Taste" caused by 'Cannot access a disposed object.' #147

Activity

1. markvantilburg commented on Mar 31, 2015

   

   markvantilburg

   on Mar 31, 2015

   Contributor

   More actions

   This still happens in the current version. I tried with the debugger attached but visual studio was not able to show what object it tries to access.
2. markvantilburg commented on Mar 31, 2015

   

   markvantilburg

   on Mar 31, 2015

   Contributor

   More actions

   Call stack according to intellitrace

   Pretzel.exe!Firefly.Utils.SocketWrapper.Shutdown(System.Net.Sockets.SocketShutdown how = {unknown}) C#
   Pretzel.exe!<>c__DisplayClass9.AnonymousMethod() C#
   Pretzel.exe!<>c__DisplayClass6.AnonymousMethod() C#
   Pretzel.exe!Firefly.Utils.SocketSender.SetStateImmediate() C#
   Pretzel.exe!Firefly.Utils.SocketSender.DoSendCompleted() C#
   Pretzel.exe!Firefly.Utils.SocketSender.SocketEventCompleted() C#
   Pretzel.exe!Firefly.Utils.SocketEventWrapper.SocketEventCompleted(object sender = {unknown}, System.Net.Sockets.SocketAsyncEventArgs e = {unknown}) C#
3. laedit commented on Mar 31, 2015

   

   laedit

   on Mar 31, 2015

   Contributor

   More actions

   This is coming from the embedded server which is no longer supported...
   We need to replace it but I haven't found any equivalent.
4. markvantilburg commented on Apr 23, 2015

   

   markvantilburg

   on Apr 23, 2015

   Contributor

   More actions

   Would it be easy to replace the server with IIS Express (if the user has that installed?)
5. markvantilburg commented on Apr 23, 2015

   

   markvantilburg

   on Apr 23, 2015

   Contributor

   More actions

   It cannot be embedded currently but we could suggest the user to install it, and use it if it's installed.

   And for example use a command line parameter to enable the use of iisexpress where taste creates an on disk folder and points iis express to there.
6. laedit commented on Apr 23, 2015

   

   laedit

   on Apr 23, 2015

   Contributor

   More actions

   I don't think having a dependency on a thing as heavy and complex as IIS Express is a good thing.
   Specially since the issue here is not blocking.
   I think it should be best to replace Firefly with maybe NHttp or juste HttpListener.
7. JakeGinnivan commented on Apr 23, 2015

   

   JakeGinnivan

   on Apr 23, 2015

   Member

   More actions

   https://www.nuget.org/packages/Nancy.Hosting.Self is another option suggested by twitter
8. laedit commented on Apr 23, 2015

   

   laedit

   on Apr 23, 2015

   Contributor

   More actions

   I don't think we need the specificity of Nancy Hosting, we can just use Microsoft.Owin.Host.HttpListener.
9. SimonCropp commented on Apr 24, 2015

   

   SimonCropp

   on Apr 24, 2015

   More actions

   i have lots of love for nancy. perhaps a little too large for this usage, although perhaps that is insignificant compared to getting something stable and well supported in
10. damianh commented on Apr 24, 2015

    

    damianh

    on Apr 24, 2015

    More actions

    👎 for depending on IIS.

    Firefly http server is well dead. NHttp doesn't support owin and looks old too.

    Use Microsoft.Owin.Host.HttpListener or Nowin. An example
11. pysco68 commented on Apr 24, 2015

    

    pysco68

    on Apr 24, 2015

    More actions

    Just saw the tweet for this issue. I'll recommend Nowin. We use it in five bigger scale intranet applications. The greatest point is support for web sockets on any platform. Plus its really fast (10000 concurrent requests per second are doable as long as the business logic is fast - on a single process)
12. damianh commented on Apr 24, 2015

    

    damianh

    on Apr 24, 2015

    More actions

    Also, since Nowin doesn't depend on Http.sys, you don't have to fup around with Http ACLs
13. laedit commented on Apr 24, 2015

    

    laedit

    on Apr 24, 2015

    Contributor

    More actions

    Thanks for the advices, I think we'll go with Nowin.
    @markvantilburg I don't have the time to work on this right now, but a PR is welcome 😃
14. laedit

    added

    Bug

    Help wanted

    on Apr 24, 2015
15. markvantilburg commented on Feb 9, 2016

    

    markvantilburg

    on Feb 9, 2016

    Contributor

    More actions

    Cannot get it to crash anymore after the change of #298
16. markvantilburg commented on Feb 9, 2016

    

    markvantilburg

    on Feb 9, 2016

    Contributor

    More actions

    Maybe #102 can also be closed with the fix of #298
17. laedit commented on Feb 9, 2016

    

    laedit

    on Feb 9, 2016

    Contributor

    More actions

    Thanks for the update.
18. laedit

    closed this as completedon Feb 9, 2016