D22209: [effects/screenshot] Disable screenshot to file on Wayland

davidedmundson added a comment.


  I don't really agree, for the same reasons that I would have happily accepted David R's patch
  
  X11 was a problem because we need to give sandboxed apps a raw X11 connection.
  On wayland we have to give sandboxed apps a raw wayland connection.
  
  It's really important to make sure there's no data leaks in the wayland connection.
  
  It's not our responsibility to do anything else. It achieves only security theater at the cost of potentially breaking things.
  
  I won't block this patch, as xdg-desktop-portal and spectacle use the other version, but I do want to state I truly dislike the trend.

REPOSITORY
  R108 KWin

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

To: graesslin, #kwin
Cc: davidedmundson, kwin, LeGast00n, sbergeron, jraleigh, fbampaloukas, GB_2, mkulinski, ragreen, jackyalcine, Pitel, iodelay, crozbo, bwowk, ZrenBot, ngraham, alexeymin, himcesjf, lesliezhai, ali-mohamed, hardening, jensreuterberg, abetts, sebas, apol, mart