D5394: KAuth integration in document saving - vol. 2

[Martin Kostolný]

martinkostolny updated this revision to Diff 13539.
martinkostolny added a comment.


  Thanks for noticing all these security issues!
  
  Both issues should now be fixed.
  
  Regarding the KAuth dialog, I agree, but couldn't find a simple way of propagating additional info to the dialog. I didn't dig in KAuth code much but so far I didn't find a way. It looks like it isn't supported. KAuth::Action.setDetails(...) doesn't really add anything in the dialog.

REPOSITORY
  R39 KTextEditor

CHANGES SINCE LAST UPDATE
  https://phabricator.kde.org/D5394?vs=13517&id=13539

REVISION DETAIL
  https://phabricator.kde.org/D5394

AFFECTED FILES
  src/buffer/katesecuretextbuffer.cpp
  src/buffer/katesecuretextbuffer_p.h
  src/buffer/katetextbuffer.cpp
  src/buffer/katetextbuffer.h

To: martinkostolny, #ktexteditor, fvogt
Cc: elvisangelaccio, aacid, ivan, lbeltrame, fvogt, apol, anthonyfieroni, cullmann, ltoscano, dhaumann, graesslin, davidedmundson, palant, kwrite-devel, dfaure, #frameworks, head7, kfunk, sars
-------------- next part --------------
An HTML attachment was scrubbed...
URL: <http://mail.kde.org/pipermail/kde-frameworks-devel/attachments/20170417/150b05b9/attachment-0001.html>