botframework-components
botframework-components copied to clipboard
Change generated csproj AspNetCoreHostingModel to be InProc (rather than OutOfProc)
Currently the templates generate a csproj that sets the AspNetCoreHostModel to OutOfProcess:

This change was first introduced as part of Update runtime template to support DL-ASE to add support for ASE, however, based on the ASP.Net Core documentation, InProc is more performant than OutOfProcess.
I think we should remove the AspNetCoreHostingModel line from the templates so all the bots are generated using InProcess by default and also update the DL ASE docs to instruct devs to change the csproj to use OutOfProcess (in addition to what they need to do to add to support named pipes).
Note: this assumes that named pipes requires OutOfProc, it may not be required anymore if they move to netcore 3.1 and we probably need to do further validation when we decide to address this task (but we still should fallback to InProc for all new bots)