> On May 7, 2013, 8:15 p.m., Àlex Fiestas wrote: > > I'd like to enable this by default, most people I know from the community > > thinks alike. > > > > Code wise it makes sense. > > Eike Hein wrote: > For clarification: Do you mean enable (= prompt, already the default) or > disable (be silent)? > > Àlex Fiestas wrote: > Silent. > > App identification on wallet are as secure as nothing, so we better have > nothing and with that a better user experience. > > Volker Krause wrote: > Yes, please!
+100000 - Harald ----------------------------------------------------------- This is an automatically generated e-mail. To reply, visit: http://git.reviewboard.kde.org/r/110330/#review32224 ----------------------------------------------------------- On May 6, 2013, 5:28 p.m., Eike Hein wrote: > > ----------------------------------------------------------- > This is an automatically generated e-mail. To reply, visit: > http://git.reviewboard.kde.org/r/110330/ > ----------------------------------------------------------- > > (Updated May 6, 2013, 5:28 p.m.) > > > Review request for KDE Runtime and Harald Sitter. > > > Description > ------- > > kwalletd has a "Prompt when an application accesses an open wallet" config > option. If this option is enabled (it is by default) any such access attempt > opens a dialog box asking the user to approve or deny the attempt, and > optionally remember the decision for the future. This patch moves the > evaluation of this config option into the codepath taken by any app > authorization check, in effect turning it into a "Prompt when an application > accesses a wallet" setting. > > The purpose is to allow distributions such as Kubuntu and Netrunner which > want to make KWallet mostly invisible during routine operations to disable > this setting by default and so avoid the user being prompted to grant > applications wallet access rights in more situations. (It should be pointed > out that application identity is apparently based on KAboutData information > anyway, and so the security of this system is dubious to begin with.) > > > In the interest of keeping the delta between upstream and downstream as small > as possible I'd say it makes sense to pick this up. > > > This diff is to be applied after the diff in: > https://git.reviewboard.kde.org/r/110328/ > > A patch rewording the checkbox label in kwalletmanager has been posted for > review here: https://git.reviewboard.kde.org/r/110331/ > > > Diffs > ----- > > kwalletd/kwalletd.cpp fa9fc11 > > Diff: http://git.reviewboard.kde.org/r/110330/diff/ > > > Testing > ------- > > Test package for Kubuntu by Harald Sitter, operation verified at runtime. > > > Thanks, > > Eike Hein > >