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.

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

Proxy slow when download file #69

Closed
nutinshell opened this issue May 23, 2013 · 7 comments
Closed

Proxy slow when download file #69

nutinshell opened this issue May 23, 2013 · 7 comments

Comments

@nutinshell
Copy link

By using Chrome with SPDY proxy, when downloading big file at high speed, browsing web pages become very slow, is this the unknown bug? Seems like issue: spdy-http2/node-spdy@be33b5a but I'm not sure.

@tatsuhiro-t
Copy link
Owner

I don't know what the node-spdy commit does either, but this issue is inherent problem of simple SPDY priority mechanism and if the priority is equal, they are raced and not interleaved nicely.
Here is the patch to make DATA frames interleave more frequently:
https://gist.github.com/tatsuhiro-t/5643313

Could you try this patch and give us your feedback?

@nutinshell
Copy link
Author

Thanks. The node-spdy commit fixed a very same issue like this, so I wonder it may happen on shrpx. I tried the patch, seems much better, but Chrome still has great web pages latency when downloading file.

@tatsuhiro-t
Copy link
Owner

Updated the patch to give lower priority streams more chance to output
https://gist.github.com/tatsuhiro-t/5643313

@nutinshell
Copy link
Author

Hi, it gets better, but still very different with TLS proxy, downloading file has no impact to web pages browsing. Maybe it's part of Chrome's issue.

@tatsuhiro-t
Copy link
Owner

Do you mean stud by TLS proxy? Then yes, it is different because TLS proxy uses multiple connections: one for downloading large file and others for web contents. On the other hand, chrome normally uses just 1 connection to SPDY proxy and all traffic goes through it.
If the patch goes well, I'll push the changes to the repo.

@nutinshell
Copy link
Author

Yes, it's stud TLS. The patch makes better, thanks.

@tatsuhiro-t
Copy link
Owner

The change was committed. Let's close the issue.

Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
None yet
Projects
None yet
Development

No branches or pull requests

2 participants