Skip to content
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.

Sign up for GitHub

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

Jump to bottom

Low speed. Large image loads really slowly. #38

Closed
wojczitsu opened this issue Jul 5, 2018 · 2 comments
Closed

Low speed. Large image loads really slowly. #38

wojczitsu opened this issue Jul 5, 2018 · 2 comments
Labels
enhancement

Comments

@wojczitsu
Copy link

wojczitsu commented Jul 5, 2018 •
edited
Loading

In my app I serve temp directory which contains large images from camera roll and html file which displays these images using <img src='...'>. Loading of one image takes around 5s so it is really slow. Shouldn't it be fast while we're using local wifi ? Is there any way to speed it up?
@yvbeek
Copy link
Member

yvbeek commented Jul 14, 2018 •
edited
Loading

How large are the images in MBs?

@yvbeek
Copy link
Member

yvbeek commented Aug 4, 2018 •
edited
Loading

I've tested downloading a large image (16.4 MB) and it completed in 225 milliseconds:

photodownload

If you download a lot of images simultaneously you will probably run into problems though. The file handler in Telegraph is kinda basic at the moment. It will load the entire file in memory, then send it to the client.

To properly fix this I need to introduce proper streaming of the HTTPMessage body.
Work is quite busy at the moment, but I'll look into it when I have the time.

I'm closing this one, because it is similar to issue #27

@yvbeek yvbeek closed this as completed Aug 4, 2018
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Assignees
No one assigned
Labels
enhancement
Projects
None yet
Milestone
No milestone
Development

No branches or pull requests
2 participants
@yvbeek @wojczitsu