Review Request 125648: Add method to run a script from DBus directly

Martin Gräßlin mgraesslin at kde.org
Mon Oct 19 14:06:41 UTC 2015


-----------------------------------------------------------
This is an automatically generated e-mail. To reply, visit:
https://git.reviewboard.kde.org/r/125648/#review87077
-----------------------------------------------------------


KWin supports loading scripts at runtime through a dbus interface. Though we don't accept the code directly, but a path to a file to load.

So from security perspective: never seems to have been a problem in KWin. And also no much difference to not providing it: install script in data dir, enabled in config, send dbus signal for KWin to reconfigure.

- Martin Gräßlin


On Oct. 15, 2015, 7:22 p.m., David Edmundson wrote:
> 
> -----------------------------------------------------------
> This is an automatically generated e-mail. To reply, visit:
> https://git.reviewboard.kde.org/r/125648/
> -----------------------------------------------------------
> 
> (Updated Oct. 15, 2015, 7:22 p.m.)
> 
> 
> Review request for Plasma.
> 
> 
> Repository: plasma-workspace
> 
> 
> Description
> -------
> 
> loadScriptInInteractiveConsole requires user interaction which blocks
> some purposes, like setting the desktop background from an app.
> 
> This runs a given script directly from a passed string without any prompts.
> 
> 
> CCBUG: 217950
> 
> ---
> 
> Questions:
> 
> I also toyed with returning the value of a script as a QDBusVariant, thoughts?
> 
> Should I not run if widgets are locked?
> 
> 
> Diffs
> -----
> 
>   shell/dbus/org.kde.PlasmaShell.xml cf2d0bbfcdb5720bf19d42403ee8c7fd542adecd 
>   shell/shellcorona.h 0e6fe041787b2f75d791b38661b63614e27d33bf 
>   shell/shellcorona.cpp 37d05347d67b7178db4d29a1ac368aacaacf0c0a 
> 
> Diff: https://git.reviewboard.kde.org/r/125648/diff/
> 
> 
> Testing
> -------
> 
> Ran this:
> var allDesktops = desktops();
> print (allDesktops);
> 
> for (i=0;i<allDesktops.length;i++) {
>     d = allDesktops[i];
>     d.wallpaperPlugin = "org.kde.image";
>     d.currentConfigGroup = Array("Wallpaper", "org.kde.image", "General");
>     d.writeConfig("Image", "file:///home/david/pictures/Wallpapers/northern_lights-wide.jpg")
> }
> 
> 
> also ran some with errors, that behaved as expected too.
> 
> 
> Thanks,
> 
> David Edmundson
> 
>

-------------- next part --------------
An HTML attachment was scrubbed...
URL: <http://mail.kde.org/pipermail/plasma-devel/attachments/20151019/3a7b715d/attachment-0001.html>


More information about the Plasma-devel mailing list