mirror of
https://github.com/opencloud-eu/opencloud.git
synced 2026-05-04 10:00:10 -05:00
Fix typos
This commit is contained in:
committed by
Ralf Haferkamp
parent
a9a3b91e80
commit
83f9db3152
@@ -57,7 +57,7 @@ func (s DrivesDriveItemService) UnmountShare(ctx context.Context, resourceID sto
|
||||
}
|
||||
|
||||
// This is a a bit of a hack. We should not rely on a specific format of the item id.
|
||||
// (But currently the ShareID is
|
||||
// But currently there is no other way to get the ShareID.
|
||||
shareId := resourceID.GetOpaqueId()
|
||||
|
||||
// Now, find out the resourceID of the shared resource
|
||||
|
||||
@@ -228,7 +228,7 @@ func cs3ReceivedSharesToDriveItems(ctx context.Context,
|
||||
// the oldest share for the resource:
|
||||
// '<sharestorageid>$<sharespaceid>!<share-opaque-id>
|
||||
// Note: This means that the driveitem ID will change when the oldest
|
||||
// shared is removed. It would be good to have are more stable ID here (e.g.
|
||||
// share is removed. It would be good to have are more stable ID here (e.g.
|
||||
// derived from the shared resource's ID. But as we need to use the same
|
||||
// ID across all services this means we needed to make similar adjustments
|
||||
// to the sharejail (usershareprovider, webdav). Which we can't currently do
|
||||
@@ -320,8 +320,8 @@ func cs3ReceivedSharesToDriveItems(ctx context.Context,
|
||||
if err != nil {
|
||||
// TODO: define a proper error behavior here. We don't
|
||||
// want the whole request to fail just because a single
|
||||
// resource owner couldn't be resolved. But, should be
|
||||
// really return the affect share in the response?
|
||||
// resource owner couldn't be resolved. But, should we
|
||||
// really return the affected share in the response?
|
||||
// For now we just log a warning. The returned
|
||||
// identitySet will just contain the userid.
|
||||
logger.Warn().Err(err).Str("userid", userID.String()).Msg("could not get owner of shared resource")
|
||||
|
||||
Reference in New Issue
Block a user