trying to play unplayable videos starts long running PHP processe

[fafische]

Describe the bug
After playing a video fails, there are PHP processes running for a long time.
The processes cause very high IO load. The process ids change permanently.
The duration the processes run, depents on the size of the file and the number failed trys to play a video. Closing browser-tab does not stop running processes.
When only a single file was accessed:
4GB mkv -> 25 seconds @350 MByte/s
8GB mkv -> 50 seconds @350 MByte/s
Disabling files_videoplayer does not change anything. When disabling viewer, files is downloaded instead of playing it.

To Reproduce
Get an unsupported video file.
Try to play it in browser. -> "Fehler beim Laden #filename#"
Check processes on server

Expected behavior
When playing a video fails, this does not cause significant server load.

Server
Archlinux (kernel 5.3.13-arch1-1)
php-fpm 7.3.12
Apache/2.4.41
Nextcloud 17.0.2
Viewer 1.2.0

Desktop (please complete the following information):

• Windows 10
• Firefox 71.0 (64-bit)

Browser log

• JQMIGRATE: Migrate is installed, version 1.4.1 jquery-migrate.min.js:2:551
• Diese Seite verwendet die nicht standardisierte Eigenschaft "zoom". Stattdessen sollte calc() in den entsprechenden Eigenschaftswerten oder "transform" zusammen mit "transform-origin: 0 0" verwendet werden. injectStylesIntoStyleTag.js:201:10
• oc_config is deprecated: use OC.config instead globals.js:26:15
• oc_requesttoken is deprecated: use OC.requestToken instead 2 globals.js:26:15
• Proxying an event bus of version 0.2.1 with 0.2.0 ProxyBus.js:30:14
• OCA.Viewer initialized Viewer.js:37:10
• New search handler registered search.js:67:12
• The escapeHTML library is deprecated! It will be removed in nextcloud 19. 37 globals.js:26:15
• The select2 library is deprecated! It will be removed in nextcloud 19. 2 globals.js:26:15
• The escapeHTML library is deprecated! It will be removed in nextcloud 19. 2 globals.js:26:15
• The humanFileSize library is deprecated! It will be removed in nextcloud 19. 31 globals.js:26:15
• HTTP-Content-Type "video/x-matroska" wird nicht unterstützt. Laden der Medienressource ##replaced##.mkv fehlgeschlagen. files
• Error loading ##replaced##.mkv
• error { target: video.hidden-visually.file-view, isTrusted: true, srcElement: video.hidden-visually.file-view, eventPhase: 0, bubbles: false, cancelable: false, returnValue: true, defaultPrevented: false, composed: false, timeStamp: 10333, … }
• Mime.js:111:11
• The escapeHTML library is deprecated! It will be removed in nextcloud 19. globals.js:26:15
• HTTP-Content-Type "video/x-matroska" wird nicht unterstützt. Laden der Medienressource ##replaced##.mkv fehlgeschlagen. files
• Error loading ##replaced##.mkv
• error { target: video.file-view.active, isTrusted: true, srcElement: video.file-view.active, eventPhase: 0, bubbles: false, cancelable: false, returnValue: true, defaultPrevented: false, composed: false, timeStamp: 10354, … }
• Mime.js:111:11
• The escapeHTML library is deprecated! It will be removed in nextcloud 19. globals.js:26:15
• HTTP-Content-Type "video/x-matroska" wird nicht unterstützt. Laden der Medienressource ##replaced##.mkv fehlgeschlagen. files
• Error loading ##replaced##.mkv
• error { target: video.hidden-visually.file-view, isTrusted: true, srcElement: video.hidden-visually.file-view, eventPhase: 0, bubbles: false, cancelable: false, returnValue: true, defaultPrevented: false, composed: false, timeStamp: 10424, … }
• Mime.js:111:11
• The escapeHTML library is deprecated! It will be removed in nextcloud 19. globals.js:26:15
• Medien konnte nicht abgespielt werden. Keine Dekoder für angefragte Formate: video/x-matroska, video/x-matroska, video/x-matroska files
• The escapeHTML library is deprecated! It will be removed in nextcloud 19. 6 globals.js:26:15
• The escapeHTML library is deprecated! It will be removed in nextcloud 19. 2 globals.js:26:15
• The escapeHTML library is deprecated! It will be removed in nextcloud 19. 2 globals.js:26:15
• The escapeHTML library is deprecated! It will be removed in nextcloud 19. 2 globals.js:26:15
• The escapeHTML library is deprecated! It will be removed in nextcloud 19. 2 globals.js:26:15
• The escapeHTML library is deprecated! It will be removed in nextcloud 19. 2 globals.js:26:15

[joshtrichards]

Looks like nextcloud/server#19568

If so, fixed in upcoming NC28.

[skjnldsv]

@joshtrichards wrong link?

[joshtrichards]

Oops. meant nextcloud/server#19567 + nextcloud/server#39560