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

Timeout-issue #71

Merged
merged 1 commit into from
Apr 25, 2014
Merged

Timeout-issue #71

merged 1 commit into from
Apr 25, 2014

Conversation

fluzzi
Copy link
Contributor

@fluzzi fluzzi commented Apr 25, 2014

esto resuelve el problema de que al pausar mas de 2 minutos el video
vuelve a empezar. #28.

En realidad no es un problema si no que el timeout por default que tiene el http server de nodejs es de 2 minutos, pasado ese tiempo hace un reset de la conección tcp. este cambio lo setea en 1 hora para evitar ese problema.

pruebenlo, saludos.

esto resuelve el problema de que al pausar mas de 2 minutos el video
vuelve a empezar.
Cuevana added a commit that referenced this pull request Apr 25, 2014
@Cuevana Cuevana merged commit b423ba2 into Cuevana:master Apr 25, 2014
@ivantodorovich
Copy link
Contributor

La verdad que si! Era algo muuuuy pero muy molesto!!

Como te decia en el otro issue, sería bueno hacerle PR a peerflix! mucha mas gente se beneficiaría de esta solucion!

@fluzzi
Copy link
Contributor Author

fluzzi commented Apr 25, 2014

groso! funciona bien entonces ;), ahora hago el pr en peerflix. @Cuevana por otro lado, se podría ir haciendo updates a la app oficial de cuevana storm como para que todos los usuarios vean los cambios que estamos haciendo.

@Cuevana
Copy link
Owner

Cuevana commented Apr 25, 2014

Sin dudas. Vamos a implementar un popup de release changes para poder informar a los usuarios de los cambios/novedades luego de cada nuevo release.

El único tema que debemos considerar es que el update al ser un file en mega u otro host, se hace un poco engorroso estar actualizando para el usuario, por eso no lanzamos releases luego de cada pequeño fix. Conviene agrupar varios y lanzar releases más grandes/importantes.

Sería bueno lograr compatibilidad para series 720p antes de tener este nuevo release, que opinan? Seguramente en estos días agreguemos un panel de preferencias para poder habilitar/deshabilitar opciones con mayor facilidad, y allí podríamos activar VLC, OpenSubtitles, etc.

@fluzzi
Copy link
Contributor Author

fluzzi commented Apr 25, 2014

Me parece perfecto!, lo del panel es importante para poder hacer varios cambios. Con respecto a series en 720p, cual es el camino que estan pensando? yo investigue como compilar el ffmpegsumo pero ni mi pc se lo banca ni se como empezar con chromium, la otra es usar VLC/otro player.
Lo del vlc lo tengo commentado en mi pc (funciona) para implementar cuando saquen lo de las config. Tambien hay que ver el aproach par esto, poner solo VLC o que el usuario pueda agregar el reprodutor que quiera como decia @ivantodorovich?

@ivantodorovich
Copy link
Contributor

El host es un tema. nodewebkit es lo más pesado del app.. el updater no debería intentar descargarlo de nuevo, es ineficiente. Si en lugar de concatenar el app.nw (zip) al ejecutable lo dejamos como archivo separado, sería mucho más facil actualizar sólo el app, y reusar el nw.

Por lo de 720p.. intenté compilar chromium, pero después de 3 horas de descargar source con gclient (y todavía no haber llegado ni al 10%, creo yo -- porque ni progreso te pone) se me cortó la luz y perdí todo. Desistí después de eso.

👍 +1 por el panel de preferencias.

Propongo reveer la estructura del app, tal vez usar Backbone.js o Vue.js.. o simplemente modulos de node.js. Separar la lógica de las vistas/templates, y modularizar en varios componentes en lugar de uno solo gigante. Va a ser dificil de mantener si seguimos incorporándole cosas así.

@Cuevana
Copy link
Owner

Cuevana commented Apr 25, 2014

Si, deberíamos migrar a una estructura más mantenible. Además de las que propone @ivantodorovich también podemos usar AngularJS o Ember.js. Comentarios?

@fluzzi fluzzi deleted the timeout-issue branch April 26, 2014 00:52
Cuevana added a commit that referenced this pull request Sep 14, 2014
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

Successfully merging this pull request may close these issues.

4 participants