flathub / com.prusa3d.PrusaSlicer

https://flathub.org/apps/details/com.prusa3d.PrusaSlicer
7 stars 14 forks source link

don't use filesystem=home #80

Closed alexmo1997 closed 10 months ago

alexmo1997 commented 11 months ago

Using filesystem=home is not really a good practice as it allows access to the whole user home directory which is not needed here.

xarbit commented 10 months ago

That is correct, the reason we use it is because PrusaSlicer and wxWidgets do not support portals API yet. How else would we provide access to the users home directory so they can load and save their projects and stl's?

alexmo1997 commented 10 months ago

Well, correct me if I'm wrong, but from another project I seem to remember that wxwidgets now does in fact support xdg portals already.

xarbit commented 10 months ago

I am not aware of this yet, would definitely be cool. We are also still waiting for proper Wayland support for wxWidgets.

Prusa also has its own fork of wxWidgets and for what its worth, they would need to update their code to support it never the less.

As soon as PrusaSlicer is there, we will certainly adopt it.

But for now I will close this issue.