Skip to content

Move the basic functionality of file streaming to a lower level API #2807

New issue

Have a question about this project? Sign up for a free GitHub account to open an issue and contact its maintainers and the community.

By clicking “Sign up for GitHub”, you agree to our terms of service and privacy statement. We’ll occasionally send you account related emails.

Already on GitHub? Sign in to your account

Closed
jbagga opened this issue Jan 19, 2018 · 5 comments
Closed

Move the basic functionality of file streaming to a lower level API #2807

jbagga opened this issue Jan 19, 2018 · 5 comments

Comments

@jbagga
Copy link
Contributor

jbagga commented Jan 19, 2018

From @jbagga on January 18, 2018 23:51

See aspnet/Mvc#7208 (comment)

@Tratcher @dustinmoris

Copied from original issue: aspnet/Mvc#7259

@jbagga
Copy link
Contributor Author

jbagga commented Jan 19, 2018

From @rynowak on January 19, 2018 0:53

This isn't an MVC issue 😆 - a lower level implies non-MVC

@dustinmoris
Copy link

Thanks, sounds like this is the right place, however I remember @DamianEdwards and @Eilon saying on one of the recent stand ups that issues in aspnet/Home are getting forgotten.

@rynowak
Copy link
Member

rynowak commented Jan 19, 2018

This is the right place. We're doing a trial of consolidating issues from lower-traffic repos in aspnet/Home.

@khellang
Copy link
Member

khellang commented Jan 26, 2018

With the advent of IActionResultExecutor<TResult>, it's pretty easy to knock out a helper extension method to execute FileStreamResult in any middleware: http://kristian.hellang.com/using-mvc-result-executors-in-middleware/.

You still need a dependency on Microsoft.AspNetCore.Mvc.Core (and the formatters you need), but it's a nice start.

@Eilon
Copy link
Contributor

Eilon commented Sep 28, 2018

Closing because there are no plans to implement this at this time.

@Eilon Eilon closed this as completed Sep 28, 2018
@ghost ghost locked as resolved and limited conversation to collaborators Dec 4, 2019
Sign up for free to subscribe to this conversation on GitHub. Already have an account? Sign in.
Labels
None yet
Projects
None yet
Development

No branches or pull requests

5 participants