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

mp4 requests slower than in chrome or other browsers (keep-alive header issue?) #1784

Closed
mjs2020 opened this issue Apr 13, 2014 · 3 comments
Closed

Comments

@mjs2020
Copy link

mjs2020 commented Apr 13, 2014

Hi,
I'm building an app with popcornjs which uses HTML5 video. I've included a ffmpegsumo.dll taken from Google Chrome to be able to decode h264. The application runs in a company intranet and videos are available on an internal NAS server. I'm facing some slowness issues in the requests especially compared to other browsers, IE11 included (!). Network conditions are the same as well as the code. However when I inspect the request headers all other browsers have a:
Connection: keep-alive
header when requesting the mp4 file but NW doesn't and I'm wondering why. Considering that the request for the file comes from an HTML5 video tag and is not a XHR request is there a way I can force NW to use that header?
Or could the problem be elsewhere?
Thanks!

Want to back this issue? Post a bounty on it! We accept bounties via Bountysource.

@xdenser
Copy link

xdenser commented Apr 13, 2014

Try NW version 0.7.5. I am still using it, as newer versions are worse as for playing video.
Though v 0.7.5. also does not use keep-alive header.
Also note - there are other issues with media files in NW - for example caching does not work. #1419

@mjs2020
Copy link
Author

mjs2020 commented Apr 14, 2014

Thanks xdenser! I'll give 0.7.5 a try and feedback.

@rogerwang
Copy link
Member

This should be working with latest version now.

In 0.13 we changed to an optimized architecture so more features can be supported, see http://nwjs.io/blog/whats-new-in-0.13/ and it's good for keeping up with Chromium upstream -- we released with Node.js v6.0 and new Chromium versions within 1 day after upstream release.

The new version would fixed many issues reported here and we're scrubbing them. This issue is closed as we believe it should be fixed. Please leave a message if it isn't and we'll reopen it.

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

3 participants