parent owner becomes owner of new folders

Signed-off-by: Jörn Friedrich Dreyer <jfd@butonic.de>

pkg/storage/fs/ocis/tree.go

@@ -28,7 +28,6 @@ import (
 	provider "github.com/cs3org/go-cs3apis/cs3/storage/provider/v1beta1"
 	"github.com/cs3org/reva/pkg/appctx"
 	"github.com/cs3org/reva/pkg/errtypes"
-	"github.com/cs3org/reva/pkg/user"
 	"github.com/google/uuid"
 	"github.com/pkg/errors"
 	"github.com/pkg/xattr"
@@ -94,27 +93,19 @@ func (t *Tree) CreateDir(ctx context.Context, node *Node) (err error) {
 	// create a directory node
 	node.ID = uuid.New().String()
 
-	// who will become the owner?
-	u, ok := user.ContextGetUser(ctx)
-	switch {
-	case ok:
-		// we have a user in context
-		err = createNode(node, u.Id)
-	case t.lu.Options.EnableHome:
-		// enable home requires a user
-		log := appctx.GetLogger(ctx)
-		log.Error().Msg("home support enabled but no user in context")
-		err = errors.Wrap(errtypes.UserRequired("userrequired"), "error getting user from ctx")
-	case t.lu.Options.Owner != "":
-		// fallback to owner?
-		err = createNode(node, &userpb.UserId{
-			OpaqueId: t.lu.Options.Owner,
-		})
-	default:
-		// fallback to parent owner?
-		err = createNode(node, nil)
+	// who will become the owner? the owner of the parent node, not the current user
+	var p *Node
+	p, err = node.Parent()
+	if err != nil {
+		return
+	}
+	var owner *userpb.UserId
+	owner, err = p.Owner()
+	if err != nil {
+		return
 	}
 
+	err = createNode(node, owner)
 	if err != nil {
 		return nil
 	}