fix share path when listing a specific share

changelog/unreleased/fix-share-path.md

@@ -0,0 +1,5 @@
+Bugfix: Fix share path when listing a specific share 
+
+The file path and target of shares where missing the share jail path when a specific share was listed. 
+
+https://github.com/cs3org/reva/pull/1918

internal/http/services/owncloud/ocs/handlers/apps/sharing/shares/shares.go

@@ -54,6 +54,7 @@ import (
 	"github.com/cs3org/reva/pkg/share"
 	"github.com/cs3org/reva/pkg/share/cache"
 	"github.com/cs3org/reva/pkg/share/cache/registry"
+	"github.com/cs3org/reva/pkg/user"
 	"github.com/cs3org/reva/pkg/utils"
 	"github.com/pkg/errors"
 )
@@ -299,6 +300,7 @@ func (h *Handler) GetShare(w http.ResponseWriter, r *http.Request) {
 		resourceID = psRes.Share.ResourceId
 	}
 
+	var receivedShare *collaboration.ReceivedShare
 	if share == nil {
 		// check if we have a user share
 		logger.Debug().Str("shareID", shareID).Msg("get user share by id")
@@ -330,10 +332,30 @@ func (h *Handler) GetShare(w http.ResponseWriter, r *http.Request) {
 
 		if err == nil && uRes.GetShare() != nil {
 			resourceID = uRes.Share.ResourceId
-			share, err = conversions.CS3Share2ShareData(ctx, uRes.Share)
-			if err != nil {
-				response.WriteOCSError(w, r, response.MetaServerError.StatusCode, "error mapping share data", err)
-				return
+			currentUser := user.ContextMustGetUser(ctx)
+			if utils.UserEqual(uRes.Share.Creator, currentUser.Id) {
+				share, err = conversions.CS3Share2ShareData(ctx, uRes.Share)
+				if err != nil {
+					response.WriteOCSError(w, r, response.MetaServerError.StatusCode, "error mapping share data", err)
+					return
+				}
+			} else {
+				rsRes, err := client.GetReceivedShare(ctx, &collaboration.GetReceivedShareRequest{
+					Ref: &collaboration.ShareReference{
+						Spec: &collaboration.ShareReference_Id{
+							Id: uRes.Share.Id,
+						},
+					},
+				})
+
+				if err == nil && rsRes.GetShare() != nil {
+					receivedShare = rsRes.GetShare()
+					share, err = conversions.CS3Share2ShareData(ctx, rsRes.Share.Share)
+					if err != nil {
+						response.WriteOCSError(w, r, response.MetaServerError.StatusCode, "error mapping share data", err)
+						return
+					}
+				}
 			}
 		}
 	}
@@ -363,7 +385,11 @@ func (h *Handler) GetShare(w http.ResponseWriter, r *http.Request) {
 		response.WriteOCSError(w, r, response.MetaServerError.StatusCode, "error mapping share data", err)
 	}
 	h.mapUserIds(ctx, client, share)
-
+	if receivedShare != nil && receivedShare.State == collaboration.ShareState_SHARE_STATE_ACCEPTED {
+		// Needed because received shares can be jailed in a folder in the users home
+		share.FileTarget = path.Join(h.sharePrefix, path.Base(info.Path))
+		share.Path = path.Join(h.sharePrefix, path.Base(info.Path))
+	}
 	response.WriteOCSSuccess(w, r, []*conversions.ShareData{share})
 }