fix attachment upload

packages/twenty-front/src/modules/activities/components/RichTextEditor.tsx

@@ -139,6 +139,32 @@ export const RichTextEditor = ({
     return getFileAbsoluteURI(result.data.uploadFile);
   };
 
+  const prepareBody = (newStringifiedBody: string) => {
+    if (!newStringifiedBody) return newStringifiedBody;
+
+    const body = JSON.parse(newStringifiedBody);
+
+    const bodyWithSignedPayload = body.map((block: any) => {
+      if (block.type !== 'image' || !block.props.url) {
+        return block;
+      }
+
+      const imageProps = block.props;
+      const imageUrl = new URL(imageProps.url);
+
+      imageUrl.searchParams.delete('token');
+
+      return {
+        ...block,
+        props: {
+          ...imageProps,
+          url: `${imageUrl.toString()}`,
+        },
+      };
+    });
+    return JSON.stringify(bodyWithSignedPayload);
+  };
+
   const handlePersistBody = useCallback(
     (activityBody: string) => {
       if (!canCreateActivity) {
@@ -148,7 +174,7 @@ export const RichTextEditor = ({
       if (!activityTitleHasBeenSet && fillTitleFromBody) {
         updateTitleAndBody(activityBody);
       } else {
-        persistBodyDebounced(activityBody);
+        persistBodyDebounced(prepareBody(activityBody));
       }
     },
     [

packages/twenty-front/src/modules/activities/files/hooks/__tests__/useUploadAttachmentFile.test.tsx

@@ -0,0 +1,14 @@
+import { computePathWithoutToken } from '../useUploadAttachmentFile';
+
+describe('computePathWithoutToken', () => {
+  it('should remove token from path', () => {
+    const input = 'https://example.com/image.jpg?token=abc123';
+    const expected = 'https://example.com/image.jpg';
+    expect(computePathWithoutToken(input)).toBe(expected);
+  });
+
+  it('should handle path without token', () => {
+    const input = 'https://example.com/image.jpg?size=large';
+    expect(computePathWithoutToken(input)).toBe(input);
+  });
+});

packages/twenty-front/src/modules/activities/files/hooks/useUploadAttachmentFile.tsx

@@ -9,6 +9,11 @@ import { CoreObjectNameSingular } from '@/object-metadata/types/CoreObjectNameSi
 import { useCreateOneRecord } from '@/object-record/hooks/useCreateOneRecord';
 import { FileFolder, useUploadFileMutation } from '~/generated/graphql';
 
+// Note: This is probably not the right way to do this.
+export const computePathWithoutToken = (attachmentPath: string): string => {
+  return attachmentPath.replace(/\?token=[^&]*$/, '');
+};
+
 export const useUploadAttachmentFile = () => {
   const currentWorkspaceMember = useRecoilValue(currentWorkspaceMemberState);
   const [uploadFile] = useUploadFileMutation();
@@ -29,9 +34,9 @@ export const useUploadAttachmentFile = () => {
       },
     });
 
-    const attachmentUrl = result?.data?.uploadFile;
+    const attachmentPath = result?.data?.uploadFile;
 
-    if (!attachmentUrl) {
+    if (!attachmentPath) {
       return;
     }
 
@@ -42,7 +47,7 @@ export const useUploadAttachmentFile = () => {
     const attachmentToCreate = {
       authorId: currentWorkspaceMember?.id,
       name: file.name,
-      fullPath: attachmentUrl,
+      fullPath: computePathWithoutToken(attachmentPath),
       type: getFileType(file.name),
       [targetableObjectFieldIdName]: targetableObject.id,
       createdAt: new Date().toISOString(),