Narrow down ReadWrite folder permission to owner

[suiso67]

#6839 seem to add group_write and others_write along with owner_write permission when setting FileSystem::FolderPermissions::ReadWrite permission to a folder, which is addressed in #6863. Please let me know if I'm missing any context.

Resolves #6863

[claucambra]

After further feedback it seems this code has deeper issues

We are investigating a broader rewrite of this right now

Thank you for your contribution, though -- we definitely have issues properly handling reassigning the correct permissions

[mgallien]

@claucambra I am not sure I would want to skip this change
It looks really weird that the desktop client is adding too much permissions
@suiso67 can you rebase ? I cannot do it missing write access to your clone

[claucambra]

@claucambra I am not sure I would want to skip this change

It looks really weird that the desktop client is adding too much permissions

@suiso67 can you rebase ? I cannot do it missing write access to your clone

Aren't these the same permissions we applied prior to the last patch, though? Those were causing even more issues

[suiso67]

@claucambra I am not sure I would want to skip this change It looks really weird that the desktop client is adding too much permissions @suiso67 can you rebase ? I cannot do it missing write access to your clone

I did rebase and force-pushed this branch, but it seems like there're other issues which I don't have context of. Can you give me issue/pr links related to this?

[bjoernv]

This PR does not seem to solve this use case: #6863 (comment)

[claucambra]

@suiso67 Upon discussion and a rethink I think we can merge this in the mean time, as setting all perms is not a viable solution and we need a better way to handle setting/restoring perms

[claucambra]

@suiso67 could you rebase once more?

[mgallien]

/backport to stable-3.13

[mgallien]

/backport to stable-3.14

[suiso67]

@suiso67 could you rebase once more?

Sorry for being late, it has been merged already.

If you don't mind, I'd like to know if there're more context about related issue you're talking about. I came here completely blind and kind of just reverted the change, and I'd like to contribute to the real issue even thought I can't guarantee anything.