flathub / com.redis.RedisInsight

https://flathub.org/apps/com.redis.RedisInsight
4 stars 1 forks source link

Change in build 104534 rejected #41

Closed flathubbot closed 5 months ago

flathubbot commented 5 months ago

A change in build 104534 has been reviewed by the Flathub team, and rejected for the following reason:

Please explain the change from home to host filesystem access. There is no information in https://github.com/flathub/com.redis.RedisInsight/pull/35

Changes

Field Old value New value
sockets ['x11'] ['fallback-x11', 'wayland', 'x11']
filesystems ['home'] ['host']
ViktarStarastsenka commented 5 months ago

the reason for changing this was to handle the following alert

Home folder read/write access
Can read and write all data in your home folder

Nothing changed in terms of features or Redis Insight behavior.

barthalion commented 5 months ago

Hey Viktar, this is not going to make it go away, host actually allows slightly more than home. There's no way to fix it unless you're using an up-to-date Electron which supports portals, and this is only true if all the app needs is specific files instead of full folders.