[ 
https://issues.apache.org/jira/browse/THRIFT-4820?page=com.atlassian.jira.plugin.system.issuetabpanels:comment-tabpanel&focusedCommentId=16792603#comment-16792603
 ] 

Kyle Smith commented on THRIFT-4820:
------------------------------------

Ah. I simply removed the dependency and confirmed it would build but didn't try 
running it through our custom ASP.NET Core middleware.

Splitting out the HTTP component into a separate package would have the benefit 
of a smaller core package plus open up support for non-Core hosting providers. 
With a dedicated package you could create a dedicated solution without 
polluting the core library. This could include an actual middleware tie-in 
(e.g. app.UseThrift("/rpc") and service.UseThrift()).

Overall I like the idea of having separate packages for heavier pieces.

> Remove unnecessary NuGet dependency from netstd
> -----------------------------------------------
>
>                 Key: THRIFT-4820
>                 URL: https://issues.apache.org/jira/browse/THRIFT-4820
>             Project: Thrift
>          Issue Type: Improvement
>          Components: netstd - Library
>    Affects Versions: 0.12.0
>            Reporter: Kyle Smith
>            Priority: Minor
>
> You should be able to remove the dependency on the heavy Microsoft.AspNetCore 
> package. THttpServerTransport doesn't utilize anything outside the 
> Microsoft.AspNetCore.Http.Abstractions package.



--
This message was sent by Atlassian JIRA
(v7.6.3#76005)

Reply via email to