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

Video playback stops once buffer underflows #4204

Closed
nakal opened this issue Jul 6, 2019 · 22 comments · Fixed by #8132
Closed

Video playback stops once buffer underflows #4204

nakal opened this issue Jul 6, 2019 · 22 comments · Fixed by #8132

Comments

@nakal
Copy link

nakal commented Jul 6, 2019

Actual behaviour

Tapping on a video, it starts to buffer and starts to play a moment later. After a few seconds the playback buffer seems to be underflowing and the video stops for ever. It's not possible to make it play anymore.

Expected behaviour

The video player should wait for buffer to fill and continue automatically, when there is enough data to play again. It should also be possible to navigate back and play the video within the available data range.

Steps to reproduce

  1. Tap on a video.

Environment data

Android version: 9

Device model: OnePlus A6003

Stock or customized system: stock

Nextcloud app version: 3.7.0 RC4

Nextcloud server version: 16.0.1 (FreeBSD)

Logs

Please tell me if you look for something particular in logs, I'll provide it. It looks like the relevant URI in the remote.php HTTP request (used for streaming?) contains sensitive information.

@nakal nakal added the bug label Jul 6, 2019
@aaronjlawson
Copy link

It appears that I am facing the same issue as well.

Android 9
Samsung Galaxy S9+
Nextcloud 16.02

@stale
Copy link

stale bot commented Aug 7, 2019

This request did not receive an update in the last 4 weeks. Please take a look again and update the issue with new details, otherwise the issue will be automatically closed in 2 weeks. Thank you!

@stale stale bot added the stale label Aug 7, 2019
@stale stale bot closed this as completed Aug 21, 2019
@Simounet
Copy link
Member

Hi there,
I'm also encountering this issue making this app unusable to play Nextcloud videos. I have to download before playing them.

  • I don't have any error in the Nextcloud log file,
  • Everything is fine from the web view,
  • I tried to use VLC for Android to play the video but it didn't even start.
    Can I send some logs from my Android app to help you?
    Have a nice day and stay safe.

@Simounet Simounet reopened this Apr 19, 2020
@stale stale bot removed the stale label Apr 19, 2020
@tobiasKaminsky
Copy link
Member

Do you use a reverse proxy?
What is your setup?

@tobiasKaminsky tobiasKaminsky added the needs info Waiting for info from user(s). Issues with this label will auto-stale. label Apr 20, 2020
@Simounet
Copy link
Member

No reverse proxy, Apache2, PHP-FPM7.3 and as I said, it's working fine through a browser.

@nakal
Copy link
Author

nakal commented Apr 20, 2020

The bug still exists. It happens when the buffer underflows at some time during playback. It seems it improved for me slightly, because when I wrote the first report the underflow happened right at the beginning quite often.

@tobiasKaminsky
Copy link
Member

@ezaquarii as you had a deep look into media player: do you know a way to improve buffer underruns?

@ezaquarii
Copy link
Collaborator

ezaquarii commented Apr 21, 2020

@tobiasKaminsky For the video player - no. Audio player has different architecture, with playback in background. We control everything there.

Video player does not use a dedicated player service - it's just loads URL into a VideoView - very basic component encapsulating MediaPlayer. We'd need to deliver something more sophisticated than that.

First line of defense would be to increase buffer size, but tbh I'd assume that the player can recover fom an underrun itsef.

We could also take a loot at ExoPlayer.

@Simounet
Copy link
Member

Can we give a debug log to help you? Don't you encounter this issue too?

@tobiasKaminsky
Copy link
Member

Strange is that VLC also does not work.
I would have assumed that VLC is better in terms of stability…

@ezaquarii
Copy link
Collaborator

ezaquarii commented Apr 22, 2020 via email

@tobiasKaminsky
Copy link
Member

Can we give a debug log to help you? Don't you encounter this issue too?

Can you create us a test account, test if the problem occurs also there and if so send the credentials to tobias at nextcloud dot com with a reference to this issue?

@stale
Copy link

stale bot commented May 22, 2020

This bug report did not receive an update in the last 4 weeks. Please take a look again and update the issue with new details, otherwise the issue will be automatically closed in 2 weeks. Thank you!

@stale stale bot added the stale label May 22, 2020
@stale stale bot closed this as completed Jun 5, 2020
@Simounet
Copy link
Member

Simounet commented Jul 1, 2020

Sorry for the late reply. Mail sent.

@Simounet Simounet reopened this Jul 1, 2020
@stale stale bot removed the stale label Jul 1, 2020
@stale
Copy link

stale bot commented Jul 29, 2020

This bug report did not receive an update in the last 4 weeks. Please take a look again and update the issue with new details, otherwise the issue will be automatically closed in 2 weeks. Thank you!

@stale stale bot added the stale label Jul 29, 2020
@Simounet
Copy link
Member

This time, this is not my fault and I still encountering this issue. Help needed. :)

@stale stale bot removed the stale label Jul 29, 2020
@stale
Copy link

stale bot commented Aug 29, 2020

This bug report did not receive an update in the last 4 weeks. Please take a look again and update the issue with new details, otherwise the issue will be automatically closed in 2 weeks. Thank you!

@stale stale bot added the stale label Aug 29, 2020
@Simounet
Copy link
Member

Simounet commented Sep 1, 2020

It's still a very annoying issue.

@stale stale bot removed the stale label Sep 1, 2020
@tobiasKaminsky tobiasKaminsky added approved and removed needs info Waiting for info from user(s). Issues with this label will auto-stale. labels Sep 2, 2020
@tobiasKaminsky
Copy link
Member

There is an experimental switch to another video player in #8132 (comment)

This apk can be installed in parallel.
It would be great if you can give this a test, if it helps in your scenario.

@Simounet
Copy link
Member

Simounet commented Mar 8, 2021

  • on the main app, I can't play any video anymore, I just can see the icon but no video playing
  • on your QA app, I can play the whole video (which is a great progress) BUT the video doesn't seems to be stopped and run in the background even if I try to play another one. I had the video playing after I closed the app and locked my phone. Maybe it's normal at the moment but I wanted to let you know @tobiasKaminsky .

@tobiasKaminsky
Copy link
Member

Thanks for testing.
Indeed this is "just" to prove if streaming / playback is better.
I did not fully tested stop/resume, etc.

@Simounet
Copy link
Member

Simounet commented Mar 8, 2021

Good to know. So it worked. Don't know if it's normal that the current official build is not working at all then.

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

Successfully merging a pull request may close this issue.

6 participants