Bug 436553

Summary:	X-KDE-Protocols fails to work as expected in some cases
Product:	[Frameworks and Libraries] frameworks-kio	Reporter:	Stefan Brüns <stefan.bruens>
Component:	general	Assignee:	KIO Bugs <kio-bugs-null>
Status:	ASSIGNED ---
Severity:	normal	CC:	a.saoutkin, auxsvr, azrdev, bugseforuns, fabian, kdelibs-bugs-null, nate, openmindead, postix
Priority:	VHI	Keywords:	regression
Version First Reported In:	git master
Target Milestone:	---
Platform:	Other
OS:	Linux
Latest Commit:		Version Fixed In:
Sentry Crash Report:

Description Stefan Brüns 2021-05-03 21:05:51 UTC

E.g. VLC specifies SMB protocol support, but this may fail for a number of reasons:

- The share is password protected. The credentials have to be provided in VLCs plugin configuration, otherwise it fails with "VLC is unable to open the MRL 'smb://<host>/path/video.mp4'. Check the log for details.", the log tells "Operation not permitted").
- SMB protocol support is packaged as a separate plugin package, e.g. https://packages.debian.org/sid/vlc-plugin-samba
 
LibreOffice has removed SMB from the list of supported protocols for the same reason:
https://cgit.freedesktop.org/libreoffice/core/commit/sysui/desktop/menus/?id=f9f9aa9873c5851da86d33ca75e937ac022206a3

Possible solutions:
- Tell upstreams to do the same as LO, remove all conditional protocols
- Provide means to state and detect conditional support

Comment 1 Stefan Brüns 2021-05-03 21:06:34 UTC

Likely related (VLC fails to open smb://):
https://bugs.kde.org/show_bug.cgi?id=75324#c135

Comment 2 Stefan Brüns 2021-05-03 21:13:12 UTC

VLC upstream desktop file:
http://git.videolan.org/?p=vlc.git;a=blob;f=share/vlc.desktop.in#l124

Comment 3 Nate Graham 2021-05-05 20:58:07 UTC

I thought we tried to detect this in kio-fuse specifically for the VLC case...

Comment 4 a.saoutkin 2021-10-03 11:30:58 UTC

This is almost certainly a regression made by changes to KIO::DesktopExecParser::resultingArguments() after fcf4529f62e15cfc891757cf372c4ef7ee7ea4d1 was committed.

Should be an easy fix, but at the moment can't commit to fixing this.

Comment 5 Nate Graham 2021-12-01 02:38:27 UTC

*** Bug 446077 has been marked as a duplicate of this bug. ***

Comment 6 Bug Janitor Service 2021-12-19 14:10:04 UTC

A possibly relevant merge request was started @ https://invent.kde.org/frameworks/kio/-/merge_requests/674