@sync-in/server 1.10.1 → 2.0.0

package/server/applications/shares/services/shares-manager.service.js

@@ -1,8 +1,4 @@
-/*
- * Copyright (C) 2012-2025 Johan Legrand <johan.legrand@sync-in.com>
- * This file is part of Sync-in | The open source file sync and share solution
- * See the LICENSE file for licensing details
- */ "use strict";
+"use strict";
 Object.defineProperty(exports, "__esModule", {
     value: true
 });
@@ -70,7 +66,10 @@ let SharesManager = class SharesManager {
             // retrieve space permissions (cached query)
             const spacePermissions = await this.spacesQueries.permissions(user.id, share.file.space.alias, share.file.space.root?.alias);
             if (!spacePermissions) {
-                this.logger.warn(`${this.setAllowedPermissions.name} - missing space permissions : ${JSON.stringify(share)}`);
+                this.logger.warn({
+                    tag: this.setAllowedPermissions.name,
+                    msg: `missing space permissions : ${JSON.stringify(share)}`
+                });
                 throw new _common.HttpException('Space not found', _common.HttpStatus.NOT_FOUND);
             }
             // compute permissions
@@ -83,12 +82,18 @@ let SharesManager = class SharesManager {
             const userId = asAdmin ? share.ownerId : user.id;
             const sharePermissions = await this.sharesQueries.permissions(userId, share.parent.alias, +user.isAdmin);
             if (!sharePermissions) {
-                this.logger.warn(`${this.setAllowedPermissions.name} - missing share permissions : ${JSON.stringify(share)}`);
+                this.logger.warn({
+                    tag: this.setAllowedPermissions.name,
+                    msg: `missing share permissions : ${JSON.stringify(share)}`
+                });
                 throw new _common.HttpException('Share not found', _common.HttpStatus.NOT_FOUND);
             }
             share.file.permissions = sharePermissions.permissions;
         } else {
-            this.logger.error(`${this.setAllowedPermissions.name} - case not handled ${JSON.stringify(share)}`);
+            this.logger.error({
+                tag: this.setAllowedPermissions.name,
+                msg: `case not handled ${JSON.stringify(share)}`
+            });
             throw new _common.HttpException('Missing information', _common.HttpStatus.BAD_REQUEST);
         }
     }
@@ -136,7 +141,10 @@ let SharesManager = class SharesManager {
                 }
                 const realPath = _nodepath.default.join(user.filesPath, createOrUpdateShareDto.file.path);
                 if (!await (0, _files.isPathExists)(realPath)) {
-                    this.logger.warn(`${this.createShare.name} - location does not exist : ${realPath}`);
+                    this.logger.warn({
+                        tag: this.createShare.name,
+                        msg: `location does not exist : ${realPath}`
+                    });
                     throw new _common.HttpException('The location does not exist', _common.HttpStatus.NOT_FOUND);
                 }
                 const fileProps = {
@@ -170,7 +178,10 @@ let SharesManager = class SharesManager {
                     throw new _common.HttpException(e.message, _common.HttpStatus.BAD_REQUEST);
                 }
                 if (!await (0, _files.isPathExists)(space.realPath)) {
-                    this.logger.warn(`${this.createShare.name} - space location does not exist : *${space.alias}* (${space.id}) : ${space.realPath}`);
+                    this.logger.warn({
+                        tag: this.createShare.name,
+                        msg: `space location does not exist : *${space.alias}* (${space.id}) : ${space.realPath}`
+                    });
                     throw new _common.HttpException('The location does not exist', _common.HttpStatus.NOT_FOUND);
                 }
                 share.spaceId = space.id;
@@ -252,7 +263,10 @@ let SharesManager = class SharesManager {
             await this.deleteAllLinkMembers(shareId, _links.LINK_TYPE.SHARE);
             await this.removeShareFromOwners(shareId, 'all', false, user.id);
         } catch (e) {
-            this.logger.error(`${this.deleteShare.name} - unable to delete share (${shareId}) (asAdmin = ${asAdmin}) : ${e}`);
+            this.logger.error({
+                tag: this.deleteShare.name,
+                msg: `unable to delete share (${shareId}) (asAdmin = ${asAdmin}) : ${e}`
+            });
             throw new _common.HttpException('Unable to delete share', _common.HttpStatus.INTERNAL_SERVER_ERROR);
         }
     }
@@ -278,15 +292,24 @@ let SharesManager = class SharesManager {
         // check parent share
         const pSharePermissions = await this.sharesQueries.permissions(user.id, createOrUpdateShareDto.parent.alias, +user.isAdmin);
         if (!pSharePermissions) {
-            this.logger.warn(`${this.createChildShare.name} - parent share does not exist or not authorized : ${createOrUpdateShareDto.parent.alias}`);
+            this.logger.warn({
+                tag: this.createChildShare.name,
+                msg: `parent share does not exist or not authorized : ${createOrUpdateShareDto.parent.alias}`
+            });
             throw new _common.HttpException('Parent share not found', _common.HttpStatus.NOT_FOUND);
         }
         if (!(0, _permissions.haveSpacePermission)(pSharePermissions, _spaces.SPACE_OPERATION.SHARE_OUTSIDE)) {
-            this.logger.warn(`${this.createChildShare.name} - is not allowed to share outside of : *${pSharePermissions.alias}* (${pSharePermissions.id})`);
+            this.logger.warn({
+                tag: this.createChildShare.name,
+                msg: `is not allowed to share outside of : *${pSharePermissions.alias}* (${pSharePermissions.id})`
+            });
             throw new _common.HttpException('You are not allowed to do this action', _common.HttpStatus.FORBIDDEN);
         }
         if (!pSharePermissions.enabled) {
-            this.logger.warn(`${this.createChildShare.name} - parent share is disabled : ${createOrUpdateShareDto.parent.alias}`);
+            this.logger.warn({
+                tag: this.createChildShare.name,
+                msg: `parent share is disabled : ${createOrUpdateShareDto.parent.alias}`
+            });
             throw new _common.HttpException('Parent share is disabled', _common.HttpStatus.FORBIDDEN);
         }
         let pShare;
@@ -294,7 +317,10 @@ let SharesManager = class SharesManager {
         if (pSharePermissions.root.externalPath) {
             const highestParentId = await this.sharesQueries.findHighestParentShare(pSharePermissions.id);
             if (!highestParentId) {
-                this.logger.warn(`${this.createChildShare.name} - unable to find the highest parent of : *${pSharePermissions.alias}* (${pSharePermissions.id})`);
+                this.logger.warn({
+                    tag: this.createChildShare.name,
+                    msg: `unable to find the highest parent of : *${pSharePermissions.alias}* (${pSharePermissions.id})`
+                });
                 throw new _common.HttpException('Parent share not found', _common.HttpStatus.NOT_FOUND);
             }
             pShare = await this.sharesQueries.shareEnv(highestParentId);
@@ -315,7 +341,10 @@ let SharesManager = class SharesManager {
         }
         // check file
         if (!await (0, _files.isPathExists)(pShareEnv.realPath)) {
-            this.logger.warn(`${this.createChildShare.name} - parent share location does not exist : ${pShareEnv.alias} (${pShareEnv.id}) : ${pShareEnv.realPath}`);
+            this.logger.warn({
+                tag: this.createChildShare.name,
+                msg: `parent share location does not exist : ${pShareEnv.alias} (${pShareEnv.id}) : ${pShareEnv.realPath}`
+            });
             throw new _common.HttpException('The location does not exist', _common.HttpStatus.NOT_FOUND);
         }
         /* Manage the case where the child share is created from the parent share itself */ // special case, the parent share is directly linked to the space root file
@@ -403,13 +432,22 @@ let SharesManager = class SharesManager {
                             ],
                             rmPermissions: action.rmPermissions
                         }
-                    ], false).catch((e)=>this.logger.error(`${this.updateSharesFromSpace.name} - ${e}`));
+                    ], false).catch((e)=>this.logger.error({
+                            tag: this.updateSharesFromSpace.name,
+                            msg: `${e}`
+                        }));
                 } else {
                     this.removeShareFromOwners(share.id, [
                         share.ownerId
-                    ], false).catch((e)=>this.logger.error(`${this.updateSharesFromSpace.name} - ${e}`));
+                    ], false).catch((e)=>this.logger.error({
+                            tag: this.updateSharesFromSpace.name,
+                            msg: `${e}`
+                        }));
                 }
-                this.logger.log(`${this.updateSharesFromSpace.name} - ${action.type} share (${share.id}) for owner ${share.ownerId} from space ${spaceId}`);
+                this.logger.log({
+                    tag: this.updateSharesFromSpace.name,
+                    msg: `${action.type} share (${share.id}) for owner ${share.ownerId} from space ${spaceId}`
+                });
             }
         }
     }
@@ -427,7 +465,10 @@ let SharesManager = class SharesManager {
                         ids: 'all',
                         rmPermissions: root.rmPermissions
                     }
-                ], false).catch((e)=>this.logger.error(`${this.updateSharesFromSpaceRoots.name} - ${e}`));
+                ], false).catch((e)=>this.logger.error({
+                        tag: this.updateSharesFromSpaceRoots.name,
+                        msg: `${e}`
+                    }));
             }
         }
         for (const rootId of toRemove){
@@ -489,7 +530,10 @@ let SharesManager = class SharesManager {
             linkGuest = await this.linksQueries.linkFromShare(user.id, linkId, spaceOrShareId, +user.isAdmin);
         }
         if (!linkGuest) {
-            this.logger.warn(`${this.getLinkFromSpaceOrShare.name} - unable to find link (${linkId}) on ${type} (${spaceOrShareId})`);
+            this.logger.warn({
+                tag: this.getLinkFromSpaceOrShare.name,
+                msg: `unable to find link (${linkId}) on ${type} (${spaceOrShareId})`
+            });
             throw new _common.HttpException('Link not found', _common.HttpStatus.NOT_FOUND);
         }
         return linkGuest;
@@ -504,7 +548,10 @@ let SharesManager = class SharesManager {
                     permissions: link.permissions
                 });
                 // notify the guest link (if email is specified)
-                this.notifyGuestLink(user, link, spaceOrShare.name, type === _links.LINK_TYPE.SHARE ? _constants.ACTION.ADD : _constants.ACTION.UPDATE).catch((e)=>this.logger.error(`${this.createOrUpdateLinksAsMembers.name} - ${e}`));
+                this.notifyGuestLink(user, link, spaceOrShare.name, type === _links.LINK_TYPE.SHARE ? _constants.ACTION.ADD : _constants.ACTION.UPDATE).catch((e)=>this.logger.error({
+                        tag: this.createOrUpdateLinksAsMembers.name,
+                        msg: `${e}`
+                    }));
             } else {
                 if (link.linkSettings) {
                     // modified link
@@ -519,7 +566,10 @@ let SharesManager = class SharesManager {
     async updateLinkFromSpaceOrShare(user, linkId, spaceOrShareId, type, createOrUpdateLinkDto, fromAPI = false) {
         const link = await this.getLinkFromSpaceOrShare(user, linkId, spaceOrShareId, type);
         if (!link) {
-            this.logger.error(`${this.updateLinkFromSpaceOrShare.name} - (${linkId}) from ${type} (${spaceOrShareId}) and user (${user.id}) was not found`);
+            this.logger.error({
+                tag: this.updateLinkFromSpaceOrShare.name,
+                msg: `(${linkId}) from ${type} (${spaceOrShareId}) and user (${user.id}) was not found`
+            });
             throw new _common.HttpException('Unable to find link', _common.HttpStatus.NOT_FOUND);
         }
         const fieldsWhiteList = [
@@ -557,7 +607,10 @@ let SharesManager = class SharesManager {
                             updateMember.permissions = (0, _shared.intersectPermissions)(shareLink.file.permissions, v);
                             this.sharesQueries.clearCachePermissions(shareLink.alias, [
                                 link.userId
-                            ]).catch((e)=>this.logger.error(`${this.updateLinkFromSpaceOrShare.name} - ${e}`));
+                            ]).catch((e)=>this.logger.error({
+                                    tag: this.updateLinkFromSpaceOrShare.name,
+                                    msg: `${e}`
+                                }));
                         }
                         break;
                     case 'language':
@@ -584,27 +637,36 @@ let SharesManager = class SharesManager {
             }
         }
         if (!Object.keys(updateUser).length && !Object.keys(updateLink).length && !Object.keys(updateShare).length && !Object.keys(updateMember).length) {
-            this.logger.warn(`${this.updateLinkFromSpaceOrShare.name} - no diff to update`);
+            this.logger.warn({
+                tag: this.updateLinkFromSpaceOrShare.name,
+                msg: `no diff to update`
+            });
             return fromAPI ? link : null;
         }
         try {
             await this.linksQueries.updateLinkFromSpaceOrShare(link, spaceOrShareId, updateUser, updateLink, updateShare, updateMember);
-            this.logger.debug(`${this.updateLinkFromSpaceOrShare.name} - link (${linkId}) updated : ${JSON.stringify({
-                ...{
-                    user: (0, _functions.anonymizePassword)(updateUser)
-                },
-                ...{
-                    link: updateLink
-                },
-                ...{
-                    share: updateShare
-                },
-                ...{
-                    member: updateMember
-                }
-            })}`);
+            this.logger.debug({
+                tag: this.updateLinkFromSpaceOrShare.name,
+                msg: `link (${linkId}) updated : ${JSON.stringify({
+                    ...{
+                        user: (0, _functions.anonymizePassword)(updateUser)
+                    },
+                    ...{
+                        link: updateLink
+                    },
+                    ...{
+                        share: updateShare
+                    },
+                    ...{
+                        member: updateMember
+                    }
+                })}`
+            });
         } catch (e) {
-            this.logger.error(`${this.updateLinkFromSpaceOrShare.name} - ${e}`);
+            this.logger.error({
+                tag: this.updateLinkFromSpaceOrShare.name,
+                msg: `${e}`
+            });
             throw new _common.HttpException('Unable to update link', _common.HttpStatus.INTERNAL_SERVER_ERROR);
         }
         if (fromAPI) {
@@ -632,7 +694,10 @@ let SharesManager = class SharesManager {
             guestLink.id = await this.usersQueries.createUserOrGuest(guestLink, _user.USER_ROLE.LINK);
             return guestLink;
         } catch (e) {
-            this.logger.error(`${this.createGuestLink.name} - unable to create guest link : ${e}`);
+            this.logger.error({
+                tag: this.createGuestLink.name,
+                msg: `unable to create guest link : ${e}`
+            });
             throw new _common.HttpException('Unable to create guest link', _common.HttpStatus.INTERNAL_SERVER_ERROR);
         }
     }
@@ -646,22 +711,34 @@ let SharesManager = class SharesManager {
     async updateSharesExternalPathQuota(shareId) {
         for (const share of (await this.sharesQueries.sharesQuotaExternalPaths(shareId))){
             if (!await (0, _files.isPathExists)(share.externalPath)) {
-                this.logger.warn(`${this.updateSharesExternalPathQuota.name} - *${share.alias}* home path does not exist`);
+                this.logger.warn({
+                    tag: this.updateSharesExternalPathQuota.name,
+                    msg: `*${share.alias}* home path does not exist`
+                });
                 continue;
             }
             const [size, errors] = await (0, _files.dirSize)(share.externalPath);
             for (const [path, error] of Object.entries(errors)){
-                this.logger.warn(`${this.updateSharesExternalPathQuota.name} - unable to get size for *${share.alias}* on ${path} : ${error}`);
+                this.logger.warn({
+                    tag: this.updateSharesExternalPathQuota.name,
+                    msg: `unable to get size for *${share.alias}* on ${path} : ${error}`
+                });
             }
             const shareQuota = {
                 storageUsage: size,
                 storageQuota: share.storageQuota
             };
-            this.sharesQueries.cache.set(`${_cache.CACHE_QUOTA_SHARE_PREFIX}-${share.id}`, shareQuota, _cache.CACHE_QUOTA_TTL).catch((e)=>this.logger.error(`${this.updateSharesExternalPathQuota.name} - share *${share.alias}* (${share.id}) : ${e}`));
+            this.sharesQueries.cache.set(`${_cache.CACHE_QUOTA_SHARE_PREFIX}-${share.id}`, shareQuota, _cache.CACHE_QUOTA_TTL).catch((e)=>this.logger.error({
+                    tag: this.updateSharesExternalPathQuota.name,
+                    msg: `share *${share.alias}* (${share.id}) : ${e}`
+                }));
             if (share.storageUsage !== shareQuota.storageUsage) {
                 this.sharesQueries.updateShare(share.id, {
                     storageUsage: shareQuota.storageUsage
-                }).then((updated)=>updated && this.logger.log(`${this.updateSharesExternalPathQuota.name} - share *${share.alias}* (${share.id}) - storage usage updated : ${shareQuota.storageUsage}`));
+                }).then((updated)=>updated && this.logger.log({
+                        tag: this.updateSharesExternalPathQuota.name,
+                        msg: `share *${share.alias}* (${share.id}) - storage usage updated : ${shareQuota.storageUsage}`
+                    }));
             }
             if (shareId) {
                 return shareQuota;
@@ -688,7 +765,10 @@ let SharesManager = class SharesManager {
             ]);
             toAdd = add.filter((m)=>{
                 if ((m.type === _member.MEMBER_TYPE.USER || m.type === _member.MEMBER_TYPE.GUEST) && !m.linkId && userIdsWhitelist.indexOf(m.id) === -1 || (m.type === _member.MEMBER_TYPE.GROUP || m.type === _member.MEMBER_TYPE.PGROUP) && groupIdsWhitelist.indexOf(m.id) === -1) {
-                    this.logger.warn(`${this.updateMembers.name} cannot add ${m.type} (${m.id}) to share *${share.alias}* (${share.id}) : not in the members whitelist`);
+                    this.logger.warn({
+                        tag: this.updateMembers.name,
+                        msg: `cannot add ${m.type} (${m.id}) to share *${share.alias}* (${share.id}) : not in the members whitelist`
+                    });
                     return false;
                 }
                 return true;
@@ -697,7 +777,10 @@ let SharesManager = class SharesManager {
         // filter links
         const toRemove = remove.filter((m)=>!m.linkId);
         // do remove links
-        this.deleteLinkMembers(remove.filter((m)=>!!m.linkId)).catch((e)=>this.logger.error(`${this.updateMembers.name} - ${e}`));
+        this.deleteLinkMembers(remove.filter((m)=>!!m.linkId)).catch((e)=>this.logger.error({
+                tag: this.updateMembers.name,
+                msg: `${e}`
+            }));
         // do update members
         const status = await this.sharesQueries.updateMembers(share.id, toAdd, (0, _functions.convertDiffUpdate)(update), toRemove);
         // lists deleted and updated members as potential share owners
@@ -725,11 +808,17 @@ let SharesManager = class SharesManager {
                 }
             }
             // clear cache &|| notify
-            this.onShareActionForMembers(share, action, members, user).catch((e)=>this.logger.error(`${this.updateMembers.name} - ${e}`));
+            this.onShareActionForMembers(share, action, members, user).catch((e)=>this.logger.error({
+                    tag: this.updateMembers.name,
+                    msg: `${e}`
+                }));
         }
         // do updates
         // remove or update potential child shares
-        this.updateMembersChildSharesPermissions(share.id, currentMembers, rmMembersChildShares, upMembersChildShares).catch((e)=>this.logger.error(`${this.updateMembers.name} - ${e}`));
+        this.updateMembersChildSharesPermissions(share.id, currentMembers, rmMembersChildShares, upMembersChildShares).catch((e)=>this.logger.error({
+                tag: this.updateMembers.name,
+                msg: `${e}`
+            }));
     }
     async updateMembersChildSharesPermissions(parentShareId, currentMembers, toRemove, toUpdate) {
         /*
@@ -756,7 +845,10 @@ let SharesManager = class SharesManager {
                 const memberGroupIds = await this.usersQueries.groupsWhitelist(m.id);
                 const groupSharePermission = currentMembers.find((m)=>(m.type === _member.MEMBER_TYPE.GROUP || m.type === _member.MEMBER_TYPE.PGROUP) && memberGroupIds.indexOf(m.id) > -1 && (0, _permissions.havePermission)(m.permissions, _spaces.SPACE_OPERATION.SHARE_OUTSIDE));
                 if (groupSharePermission) {
-                    this.logger.debug(`${this.diffSharesPermissions.name} - ignore user (${m.id}) removal : is a member of the group (${groupSharePermission.id}) with share permission`);
+                    this.logger.debug({
+                        tag: this.diffSharesPermissions.name,
+                        msg: `ignore user (${m.id}) removal : is a member of the group (${groupSharePermission.id}) with share permission`
+                    });
                     continue;
                 }
                 removeUsersChildShares.push(m.id);
@@ -889,12 +981,18 @@ let SharesManager = class SharesManager {
                     share.ownerId
                 ], null, false);
                 await this.sharesQueries.deleteShare(share.id);
-                this.logger.log(`${this.removeShareFromOwners.name} - share *${share.alias}* (${share.id}) from owner (${share.ownerId}) was removed`);
+                this.logger.log({
+                    tag: this.removeShareFromOwners.name,
+                    msg: `share *${share.alias}* (${share.id}) from owner (${share.ownerId}) was removed`
+                });
                 // clear cache and notify users
                 if (share.ownerId && (!fromUserId || fromUserId !== share.ownerId)) {
                     this.clearCachePermissionsAndOrNotify(share, _constants.ACTION.DELETE_PERMANENTLY, [
                         share.ownerId
-                    ]).catch((e)=>this.logger.error(`${this.removeShareFromOwners.name} - ${e}`));
+                    ]).catch((e)=>this.logger.error({
+                            tag: this.removeShareFromOwners.name,
+                            msg: `${e}`
+                        }));
                 }
                 members.forEach((m)=>this.clearCachePermissionsAndOrNotify({
                         alias: m.shareAlias,
@@ -903,7 +1001,10 @@ let SharesManager = class SharesManager {
                         m.userId
                     ]));
             } catch (e) {
-                this.logger.error(`${this.removeShareFromOwners.name} - share *${share.alias}* (${share.id}) from owner (${share.ownerId}) was not removed : ${e}`);
+                this.logger.error({
+                    tag: this.removeShareFromOwners.name,
+                    msg: `share *${share.alias}* (${share.id}) from owner (${share.ownerId}) was not removed : ${e}`
+                });
             }
         }
     }
@@ -925,10 +1026,19 @@ let SharesManager = class SharesManager {
                         name: m.shareName
                     }, _constants.ACTION.UPDATE, [
                         m.userId
-                    ]).catch((e)=>this.logger.error(`${this.removeChildSharesPermissions.name} - ${e}`));
-                    this.logger.log(`${this.removeChildSharesPermissions.name} - user (${m.id}) permissions ${JSON.stringify(userPerm.rmPermissions)} on share : ${m.shareAlias} (${m.shareId}) was removed`);
+                    ]).catch((e)=>this.logger.error({
+                            tag: this.removeChildSharesPermissions.name,
+                            msg: `${e}`
+                        }));
+                    this.logger.log({
+                        tag: this.removeChildSharesPermissions.name,
+                        msg: `user (${m.id}) permissions ${JSON.stringify(userPerm.rmPermissions)} on share : ${m.shareAlias} (${m.shareId}) was removed`
+                    });
                 } catch (e) {
-                    this.logger.error(`${this.removeChildSharesPermissions.name} - user (${m.id}) permissions on share *${m.shareAlias}* (${m.shareId}) was not removed : ${e}`);
+                    this.logger.error({
+                        tag: this.removeChildSharesPermissions.name,
+                        msg: `user (${m.id}) permissions on share *${m.shareAlias}* (${m.shareId}) was not removed : ${e}`
+                    });
                 }
             }
         }
@@ -936,7 +1046,10 @@ let SharesManager = class SharesManager {
     async checkChildSharePermissions(user, shareId, childId) {
         const isOwnerForChildShare = await this.sharesQueries.childExistsForShareOwner(user.id, shareId, childId, user.isAdmin);
         if (isOwnerForChildShare !== childId) {
-            this.logger.warn(`${this.checkChildSharePermissions.name} - is not allowed to manage child share (${childId}) from share (${shareId})`);
+            this.logger.warn({
+                tag: this.checkChildSharePermissions.name,
+                msg: `is not allowed to manage child share (${childId}) from share (${shareId})`
+            });
             throw new _common.HttpException('Not authorized', _common.HttpStatus.FORBIDDEN);
         }
         return true;
@@ -945,16 +1058,25 @@ let SharesManager = class SharesManager {
         this.clearCachePermissionsAndOrNotify(share, action, Array.from(new Set([
             ...await this.usersQueries.allUserIdsFromGroupsAndSubGroups(members.groupIds),
             ...members.userIds
-        ])).filter((uid)=>uid !== user?.id), user).catch((e)=>this.logger.error(`${this.onShareActionForMembers.name} - ${e}`));
+        ])).filter((uid)=>uid !== user?.id), user).catch((e)=>this.logger.error({
+                tag: this.onShareActionForMembers.name,
+                msg: `${e}`
+            }));
     }
     async createLinkFromSpaceOrShare(user, uuid, spaceOrShareId, type, createOrUpdateLinkDto) {
         /* only used during the share creation from this manager */ if (!await this.linksQueries.isReservedUUID(user.id, uuid)) {
-            this.logger.error(`${this.createLinkFromSpaceOrShare.name} - user attempted to use UUID (${uuid}) was not reserved`);
+            this.logger.error({
+                tag: this.createLinkFromSpaceOrShare.name,
+                msg: `user attempted to use UUID (${uuid}) was not reserved`
+            });
             throw new _common.HttpException('UUID was not reserved', _common.HttpStatus.BAD_REQUEST);
         }
         const permission = type === _links.LINK_TYPE.SPACE ? _user.GUEST_PERMISSION.SPACES : _user.GUEST_PERMISSION.SHARES;
         const guestLink = await this.createGuestLink(permission, createOrUpdateLinkDto.password, createOrUpdateLinkDto.language, createOrUpdateLinkDto.isActive !== undefined ? createOrUpdateLinkDto.isActive : true);
-        this.logger.debug(`${this.createLinkFromSpaceOrShare.name} - guest link (${guestLink.id}) created`);
+        this.logger.debug({
+            tag: this.createLinkFromSpaceOrShare.name,
+            msg: `guest link (${guestLink.id}) created`
+        });
         let linkId;
         try {
             linkId = await this.linksQueries.createLinkToSpaceOrShare(guestLink.id, spaceOrShareId, type, {
@@ -962,9 +1084,15 @@ let SharesManager = class SharesManager {
                 uuid: uuid,
                 userId: guestLink.id
             });
-            this.logger.debug(`${this.createLinkFromSpaceOrShare.name} - link (${linkId}) for guest link (${guestLink.id}) created : ${JSON.stringify(createOrUpdateLinkDto)}`);
+            this.logger.debug({
+                tag: this.createLinkFromSpaceOrShare.name,
+                msg: `link (${linkId}) for guest link (${guestLink.id}) created : ${JSON.stringify(createOrUpdateLinkDto)}`
+            });
         } catch (e) {
-            this.logger.error(`${this.createLinkFromSpaceOrShare.name} - unable to create link with uuid (${uuid}) : ${e}`);
+            this.logger.error({
+                tag: this.createLinkFromSpaceOrShare.name,
+                msg: `unable to create link with uuid (${uuid}) : ${e}`
+            });
             throw new _common.HttpException('Unable to update link', _common.HttpStatus.INTERNAL_SERVER_ERROR);
         }
     }
@@ -972,9 +1100,15 @@ let SharesManager = class SharesManager {
         for (const guestLink of guestLinks){
             try {
                 await this.usersQueries.deleteGuestLink(guestLink.id);
-                this.logger.log(`${this.deleteGuestLinks.name} - guest (${guestLink.id}) (link: ${guestLink.linkId}) was removed`);
+                this.logger.log({
+                    tag: this.deleteGuestLinks.name,
+                    msg: `guest (${guestLink.id}) (link: ${guestLink.linkId}) was removed`
+                });
             } catch (e) {
-                this.logger.error(`${this.deleteGuestLinks.name} - guest (${guestLink.id}) (link: ${guestLink.linkId}) was not removed : ${e}`);
+                this.logger.error({
+                    tag: this.deleteGuestLinks.name,
+                    msg: `guest (${guestLink.id}) (link: ${guestLink.linkId}) was not removed : ${e}`
+                });
             }
         }
     }
@@ -982,10 +1116,16 @@ let SharesManager = class SharesManager {
         if (!memberIds?.length) {
             return;
         }
-        this.logger.verbose(`${this.clearCachePermissionsAndOrNotify.name} - share:${share.alias} ${action} members:${JSON.stringify(memberIds)}`);
+        this.logger.verbose({
+            tag: this.clearCachePermissionsAndOrNotify.name,
+            msg: `share:${share.alias} ${action} members:${JSON.stringify(memberIds)}`
+        });
         if (action !== _constants.ACTION.ADD) {
             // clear permissions for share members
-            this.sharesQueries.clearCachePermissions(share.alias, memberIds).catch((e)=>this.logger.error(`${this.clearCachePermissionsAndOrNotify.name} - ${e}`));
+            this.sharesQueries.clearCachePermissions(share.alias, memberIds).catch((e)=>this.logger.error({
+                    tag: this.clearCachePermissionsAndOrNotify.name,
+                    msg: `${e}`
+                }));
         }
         if (action !== _constants.ACTION.UPDATE) {
             // notify the members who have joined or left the share
@@ -999,7 +1139,10 @@ let SharesManager = class SharesManager {
                 currentUrl: this.contextManager.headerOriginUrl(),
                 author: user,
                 action: action
-            }).catch((e)=>this.logger.error(`${this.clearCachePermissionsAndOrNotify.name} - ${e}`));
+            }).catch((e)=>this.logger.error({
+                    tag: this.clearCachePermissionsAndOrNotify.name,
+                    msg: `${e}`
+                }));
         }
     }
     async notifyGuestLink(user, link, spaceOrShareName, action) {
@@ -1024,7 +1167,10 @@ let SharesManager = class SharesManager {
             linkPassword: link.linkSettings.password,
             currentUrl: this.contextManager.headerOriginUrl(),
             action: action
-        }).catch((e)=>this.logger.error(`${this.notifyGuestLink.name} - ${e}`));
+        }).catch((e)=>this.logger.error({
+                tag: this.notifyGuestLink.name,
+                msg: `${e}`
+            }));
     }
     constructor(contextManager, notificationsManager, sharesQueries, spacesQueries, usersQueries, linksQueries){
         this.contextManager = contextManager;