Use the usermountcache to get all users who have access to a file

[icewind1991]

Fixes activity when multiple users have access to the same files not trough sharing (but instead external storage, group folders, etc)

• requires Allow getting the filepath when getting cached mounts by fileid server#6220

Fix#264

[nickvergessen]

Tested it and sharing still works.

However when using external storage, the activity is still only generated for the actor and not all other users of the external storage. Isn't that what you were trying to fix? @icewind1991

[icewind1991]

Works for me, created an external storage, gave 2 users access.

If user A uploads a file, it shows in user B's activity

[mc9625]

Files uploaded in a groupfolder are not listed in the activities.

[icewind1991]

rebased and re-tested, works for me with both groupfolders and external storage.

Note that after adding the groupfolder/external storage all involved users will need to load the files app/dav once to update the cached list of mounts the user has access to

[nickvergessen]

Okay, so I tested it a bit, moving/renaming is not working, but this can be fixed by moving your new code to the getUserPathsFromPath method. Also the paths now include /<userid>/files but that is also easily fixable.

But I'm not sure the "until they accessed webdav" is okay. The problem is that it also works the other way around. I shared a the external storage with user2, logged in+out as user2, unshared the external storage from user2 and created activity. Although user2 has no access anymore, all activity is generated and visible until they relogin/use webdav.

[putt1ck]

How does this get progressed?

[MorrisJobke]

Looks like we need to move this to 14.

[putt1ck]

the missing activity in group folders is quite an issue for clients; is this a patch we can apply to an install of 12 or 13?

[pascalBokBok]

Not having Activity for group folders gave a lot of extra work for us to track down which user had written 0KB into some files so we could restore them from that users revision.

[putt1ck]

Is there anything we can do to move this forward?

[MorrisJobke]

@nickvergessen@icewind1991 What is missing here?

[nickvergessen]

A solution to the problem so it works in both cases

[MorrisJobke]

@nickvergessen Moved to 15.

[noresistence]

Are there still plans to integrate activity in groupfolders in the activity stream of users with access to that folder?

[hriekehof]

please fix this :D it would help us so tremendous.

[ghosttable]

Hi folks,
i know you are out of ideas but this needs to get fixed.
Notification for new files inside a company Nextcloud configured with Group folders is very important for us. And i think, if you follow the Threads not only for us.

Greetz and a happy new Year from here :)

[nickvergessen]

I'm locking the conversation here. We know this is an issue, it will get fixed at some point.

[nickvergessen]

Added a config flag together with the following documentation for now:

[nickvergessen]

/backport to stable18

[nickvergessen]

So lets get this in? @icewind1991@rullzer

[daspuppy]

Quick question: Does the nextcloud client suffice as a login for this?

Otherwise I can't wait for this. Will implement it manually by the weekend if it's not merged by then. You will have made so many people happy!

[icewind1991]

👍

[wiswedel]

/backport to stable18

[wiswedel]

Is backporting not possible? Why does backport-bot not create the PR?

[nickvergessen]

I'm pretty sure backporting this code works, while the used APIs might not be available. So don't merge a potential backport immediately

[nickvergessen]

The backport was created in #457 btw, seems like the linking failed.