Skip to content
New issue

Have a question about this project? Sign up for a free GitHub account to open an issue and contact its maintainers and the community.

By clicking “Sign up for GitHub”, you agree to our terms of service and privacy statement. We’ll occasionally send you account related emails.

Already on GitHub? Sign in to your account

[Sharing] Avoid loop shares #5491

Closed
dragotin opened this issue Oct 23, 2013 · 9 comments
Closed

[Sharing] Avoid loop shares #5491

dragotin opened this issue Oct 23, 2013 · 9 comments

Comments

@dragotin
Copy link
Contributor

Consider this szenario:

User Tom shares a directory with the two users Betty and John. 
Now these both decide to share this folder with Carl. So the 
same shared folder would appear twice on Carls data set. 

That must not happen. The server has to check if the shared folder is already there at Carl.

@DeepDiver1975
Copy link
Member

@dragotin with respect to syncing is that a critical scenario which needs to be fixed in oc6? Or can this one wait until oc7? THX

@dragotin
Copy link
Contributor Author

I think it is ok to handle that with oC7.

@nickvergessen
Copy link
Contributor

@schiesbn We talked about circular sharing last week? Is this going to be resolved with that?

@PVince81
Copy link
Contributor

I think at the moment in OC 7 the folder would appear twice with different names, which I believe is acceptable for now.

In OC 8 we'll look into removing reshare chains and make it a single participant list, as stated here #9058 so this issue might be obsolete at that point.

@PVince81 PVince81 modified the milestones: ownCloud 8, ownCloud 7 Jun 20, 2014
@DeepDiver1975 DeepDiver1975 modified the milestones: 8.1-next, ownCloud 8 Jan 9, 2015
@PVince81
Copy link
Contributor

PVince81 commented Jan 9, 2015

Just observed a similar case today on master:

  1. user1 shares "test" with user2
  2. user2 shares "test" with user3
  3. user3 shares "test" with user1

Now user1 has two folders "test" and "test (2)" with the same contents.
If you move then rename "test (2)" you can use it almost like a symlink/hardlink 😉

@PVince81
Copy link
Contributor

PVince81 commented Jan 9, 2015

There was a discussion to change how reshare works: #9058
Basically have a single list of people who have access to the folder instead of having a reshare chain.

@PVince81
Copy link
Contributor

PVince81 commented Feb 4, 2015

I suggest to defer this to after #9058 is implemented (possibly 8.2)

@DeepDiver1975 DeepDiver1975 modified the milestones: 8.2-next, 8.1-current Mar 2, 2015
@MorrisJobke
Copy link
Contributor

#9058 was moved to 9.0 -> move this too.

@MorrisJobke MorrisJobke modified the milestones: 9.0-next, 8.2-current Aug 26, 2015
@rullzer rullzer mentioned this issue Oct 13, 2015
43 tasks
@DeepDiver1975
Copy link
Member

reshares will die in 9.0 -> closing. THX @dragotin

@lock lock bot locked as resolved and limited conversation to collaborators Aug 7, 2019
Sign up for free to subscribe to this conversation on GitHub. Already have an account? Sign in.
Projects
None yet
Development

No branches or pull requests

5 participants