search

tchx84 / Flatseal

Manage Flatpak permissions
GNU General Public License v3.0
1.1k stars 72 forks source link

rename misleading "Inter-process communications" toggle which is showing low-level namespace naming to end users #666

Open thestinger opened 3 months ago

thestinger commented 3 months ago

This "Inter-process communications" toggle controls whether a fresh System V IPC namespace is created, not interprocess communications as a whole. It should be renamed to something like "Legacy inter-process communications". It's not needed in practice. It's not officially deprecated or anything like that but it would be incredibly odd for it to be used for communication between user-facing applications. It's entirely disabled on some platforms and isn't something people miss in practice. It can certainly be considered legacy for desktop apps where control over D-Bus communication is likely to become finer-grained but not this, which simply isn't designed for it.

https://man7.org/linux/man-pages/man7/ipc_namespaces.7.html https://man7.org/linux/man-pages/man7/sysvipc.7.html

rusty-snake commented 3 months ago

It's basicly only required when you use X11. And even their it depends on something.

And required for X11 means things like window shadows might be rendered all black and maybe some performance drops.

tchx84 commented 3 months ago

@thestinger would you mind sending a PR with your suggestions, plus clarify this on the documentation?