Large static file cause 'Out Of Memory' error about embedio HOT 17 CLOSED

We will look into implementing large, chunked file transfers this week. Thanks for the good bug report!

from embedio.

@egops are you using HTTP partials? Usually if you don't use partial to chunk your file in multiple small request the entire file is loaded to stream it.

In the unit tests is a sample how to use partials: https://github.com/unosquare/embedio/blob/master/Unosquare.Labs.EmbedIO.Tests/StaticFilesModuleTest.cs#L108

from embedio.

@geoperez Thank you for your advice, I need the file can be downoaded by any browser, I think this solution is not suit for me.

from embedio.

@mariodivece OK, thank you for nice work.

from embedio.

@geoperez see if it would be possible to send out the file from the stream little by little avoiding an oomex please. Partials are not really the solution to this exception.

from embedio.

Sure, let me take a look.

On Sep 6, 2016 6:17 AM, "Mario Di Vece" [email protected] wrote:

@geoperez https://github.com/geoperez see if it would be possible to
send out the file from the stream little by little avoiding an oomex
please. Partials are not really the solution to this exception.

—
You are receiving this because you were mentioned.
Reply to this email directly, view it on GitHub
#42 (comment),
or mute the thread
https://github.com/notifications/unsubscribe-auth/ABsYsEpw5FBXSy4ABJT77faeGIPZGsM7ks5qnUvPgaJpZM4J08aq
.

from embedio.

@egops we continue working on this. it should be ready in a couple of days. @geoperez decided to cleanup the monolithic-looking request handler that includes large file serving

from embedio.

@egops can you check the new Nuget (1.0.23)?

from embedio.

@geoperez I updated the version, any file can NOT be downloaded by my browsers ( IE and Firefox ) now!
Logs here when I try to download file:

2016-09-12 19:43:08.419 Start of Request d8741d92
2016-09-12 19:43:08.419 Source ::1:54160 - GET: /nvio.npg
2016-09-12 19:43:08.419 Static Files Module::StaticFilesModule.<.ctor>b__37_2
2016-09-12 19:43:08.419 File System: D:/nvio.npg
2016-09-12 19:43:08.601 Result: True
2016-09-12 19:43:08.601 End of Request d8741d92

but the browser shows nothing.
Can you check it out?

from embedio.

I didn't see any issue in the unit tests, let me investigate it.

On Sep 12, 2016 6:49 AM, "egops" [email protected] wrote:

@geoperez https://github.com/geoperez I updated the version, any file
can NOT be downloaded by my browsers ( IE and Firefox ) now!
Logs here when I try to download file:

2016-09-12 19:43:08.419 Start of Request d8741d92
2016-09-12 19:43:08.419 Source ::1:54160 - GET: /nvio.npg
2016-09-12 19:43:08.419 Static Files Module::StaticFilesModule.<.ctor>b__37_2
2016-09-12 19:43:08.419 File System: D:/nvio.npg
2016-09-12 19:43:08.601 Result: True
2016-09-12 19:43:08.601 End of Request d8741d92

but the browser shows nothing.
Can you check it out?

—
You are receiving this because you were mentioned.
Reply to this email directly, view it on GitHub
#42 (comment),
or mute the thread
https://github.com/notifications/unsubscribe-auth/ABsYsFYopCt6aAAugHEqfaBdh_ed480_ks5qpTxfgaJpZM4J08aq
.

from embedio.

Guys, egops has the wrong path. See:
GET: /nvio.npg
Shouldn't it be nvio.png?
If not then verify the mime type is in the list.

from embedio.

Mario there's something wrong, I'm checking it.

On Sep 12, 2016 8:39 AM, "Mario Di Vece" [email protected] wrote:

Guys, egops has the wrong path. See:
GET: /nvio.npg
Shouldn't it be nvio.png?
If not then verify the mime type is in the list.

—
You are receiving this because you were mentioned.
Reply to this email directly, view it on GitHub
#42 (comment),
or mute the thread
https://github.com/notifications/unsubscribe-auth/ABsYsHTUSKIWYBA-hxpPPcnO61eO2SBqks5qpVYFgaJpZM4J08aq
.

from embedio.

Ok. The other strange thing I noticed is that the result was TRUE for the request. Shouldn't it return FALSE when the file is not found?

from embedio.

I found the issue, when the browser request includes Header Compression the stream compressed is not valid. Working on patch.

from embedio.

Hello @egops can you try with the new nuget?

from embedio.

@geoperez @mariodivece thank you very much, I will try it tomorrow.

from embedio.

@geoperez @mariodivece it works well now.

from embedio.