cozy-sharing 11.0.0 → 11.0.1

package/CHANGELOG.md

@@ -3,6 +3,17 @@
 All notable changes to this project will be documented in this file.
 See [Conventional Commits](https://conventionalcommits.org) for commit guidelines.
 
+## [11.0.1](https://github.com/cozy/cozy-libs/compare/cozy-sharing@11.0.0...cozy-sharing@11.0.1) (2024-03-12)
+
+
+### Bug Fixes
+
+* **sharing:** Handle memory leak in self removal ([f9a9384](https://github.com/cozy/cozy-libs/commit/f9a9384fabb8892e35be0a270452ee501bd6b1a6))
+
+
+
+
+
 # [11.0.0](https://github.com/cozy/cozy-libs/compare/cozy-sharing@10.4.1...cozy-sharing@11.0.0) (2024-03-11)
 
 

package/dist/components/Avatar/MemberAvatar.js

@@ -1,16 +1,34 @@
 import _extends from "@babel/runtime/helpers/extends";
+import _defineProperty from "@babel/runtime/helpers/defineProperty";
 import _objectWithoutProperties from "@babel/runtime/helpers/objectWithoutProperties";
 var _excluded = ["recipient"];
+
+function ownKeys(object, enumerableOnly) { var keys = Object.keys(object); if (Object.getOwnPropertySymbols) { var symbols = Object.getOwnPropertySymbols(object); enumerableOnly && (symbols = symbols.filter(function (sym) { return Object.getOwnPropertyDescriptor(object, sym).enumerable; })), keys.push.apply(keys, symbols); } return keys; }
+
+function _objectSpread(target) { for (var i = 1; i < arguments.length; i++) { var source = null != arguments[i] ? arguments[i] : {}; i % 2 ? ownKeys(Object(source), !0).forEach(function (key) { _defineProperty(target, key, source[key]); }) : Object.getOwnPropertyDescriptors ? Object.defineProperties(target, Object.getOwnPropertyDescriptors(source)) : ownKeys(Object(source)).forEach(function (key) { Object.defineProperty(target, key, Object.getOwnPropertyDescriptor(source, key)); }); } return target; }
+
 import React from 'react';
 import { useClient } from 'cozy-client';
 import Avatar from 'cozy-ui/transpiled/react/Avatar';
 import { getDisplayName, getInitials } from '../../models';
+import logger from '../../logger';
 
 var MemberAvatar = function MemberAvatar(_ref) {
   var recipient = _ref.recipient,
       rest = _objectWithoutProperties(_ref, _excluded);
 
-  var client = useClient();
+  var client = useClient(); // There are cases when due to apparent memory leaks, the recipient does not exist
+  // This will trigger an unhanded error in the Avatar component and crash the app
+  // At the moment, we are not sure where is the root cause of this cascading undefined props
+  // Nevertheless, we can prevent the crash by returning null if the recipient is undefined
+
+  if (!recipient) {
+    // eslint-disable-next-line
+    logger.warn('RecipientAvatar: recipient is missing, see props:', _objectSpread({
+      recipient: recipient
+    }, rest));
+    return null;
+  }
   /**
    * avatarPath is always the same for a recipient, but image
    * can be different since the stack generate it on the fly.
@@ -21,6 +39,7 @@ var MemberAvatar = function MemberAvatar(_ref) {
    * status changes
    */
 
+
   var image = recipient.avatarPath && recipient.status ? "".concat(client.options.uri).concat(recipient.avatarPath, "?v=").concat(recipient.status) : null;
   return /*#__PURE__*/React.createElement(Avatar, _extends({
     image: image,

package/package.json

@@ -1,6 +1,6 @@
 {
   "name": "cozy-sharing",
-  "version": "11.0.0",
+  "version": "11.0.1",
   "description": "Provides sharing login for React applications.",
   "main": "dist/index.js",
   "author": "Cozy",
@@ -71,5 +71,5 @@
   "sideEffects": [
     "*.css"
   ],
-  "gitHead": "b98230a4b8974af2699385bbe8edf5635a46b4a3"
+  "gitHead": "0ac36e39dfef446f11daaa97925814447f7d8201"
 }

package/src/components/Avatar/MemberAvatar.jsx

@@ -4,10 +4,21 @@ import { useClient } from 'cozy-client'
 import Avatar from 'cozy-ui/transpiled/react/Avatar'
 
 import { getDisplayName, getInitials } from '../../models'
+import logger from '../../logger'
 
 const MemberAvatar = ({ recipient, ...rest }) => {
   const client = useClient()
 
+  // There are cases when due to apparent memory leaks, the recipient does not exist
+  // This will trigger an unhanded error in the Avatar component and crash the app
+  // At the moment, we are not sure where is the root cause of this cascading undefined props
+  // Nevertheless, we can prevent the crash by returning null if the recipient is undefined
+  if (!recipient) {
+    // eslint-disable-next-line
+    logger.warn('RecipientAvatar: recipient is missing, see props:', { recipient, ...rest })
+    return null
+  }
+
   /**
    * avatarPath is always the same for a recipient, but image
    * can be different since the stack generate it on the fly.