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

Marco Martin notmart at gmail.com
Mon Oct 19 11:29:10 UTC 2015



> On Oct. 19, 2015, 10:43 a.m., Marco Martin wrote:
> > is true that for a shell script would already be possible to poke with the config file..
> > but wouldn't this be a bit too much of a security hole/open invitation for anybody, script or app to poke in the desktop configuration without the user being aware of it?
> 
> David Edmundson wrote:
>     How is this any different to the look and feel scripting you've literally just pushed?

It continues to rub me a bit the wrong way..
but let's try this, we can eventually add a bit of treatment of input if need arises.


- Marco


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


On Oct. 15, 2015, 5: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, 5: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/b3ae8e53/attachment.html>


More information about the Plasma-devel mailing list