Review Request: Plasmate: when "Enable signing plasmoid" is enabled, automatically choose a key

Aaron J. Seigo aseigo at kde.org
Mon May 28 13:09:13 UTC 2012


-----------------------------------------------------------
This is an automatically generated e-mail. To reply, visit:
http://git.reviewboard.kde.org/r/105074/#review14229
-----------------------------------------------------------


given that this is a security feature, this looks a little too dangerous and lose.

instead, disable the signing button altogether until there is at least one key selected.

- Aaron J. Seigo


On May 28, 2012, 8:01 a.m., Giorgos Tsiapaliwkas wrote:
> 
> -----------------------------------------------------------
> This is an automatically generated e-mail. To reply, visit:
> http://git.reviewboard.kde.org/r/105074/
> -----------------------------------------------------------
> 
> (Updated May 28, 2012, 8:01 a.m.)
> 
> 
> Review request for Plasma, Sebastian Kügler and Aaron J. Seigo.
> 
> 
> Description
> -------
> 
> Hello,
> 
> right now when "Enable signing plasmoid" is enabled if the user doesn't choose a gpg key,
> plasmate won't have a key enabled. So "Enable signing plasmoid", will be enabled but no signing will
> happen. 
> 
> How to test the issue,
> - rm ${KDEHOME}/share/config/plasmaterc
> -create a gpg key(gpg --gen-key)
> -open plasmate
> -click publisher
> -click enable
> 
> 
> Diffs
> -----
> 
>   publisher/signingwidget.cpp e63dbbd 
> 
> Diff: http://git.reviewboard.kde.org/r/105074/diff/
> 
> 
> Testing
> -------
> 
> It works.
> 
> 
> Thanks,
> 
> Giorgos Tsiapaliwkas
> 
>

-------------- next part --------------
An HTML attachment was scrubbed...
URL: <http://mail.kde.org/pipermail/plasma-devel/attachments/20120528/501a3080/attachment.html>


More information about the Plasma-devel mailing list