From 11fbf79ab719f96b2544abf7ad5cc77b0a9353d4 Mon Sep 17 00:00:00 2001 From: Audric Ackermann Date: Thu, 20 May 2021 12:06:34 +1000 Subject: [PATCH 01/11] switch to dedicated server (#1646) --- ts/components/session/ActionsPanel.tsx | 16 +++++++++++ ts/fileserver/FileServerApiV2.ts | 31 +++++++++++++++++----- ts/opengroup/opengroupV2/OpenGroupAPIV2.ts | 2 +- ts/receiver/attachments.ts | 5 ++-- 4 files changed, 44 insertions(+), 10 deletions(-) diff --git a/ts/components/session/ActionsPanel.tsx b/ts/components/session/ActionsPanel.tsx index 5ced06163..f7cb86005 100644 --- a/ts/components/session/ActionsPanel.tsx +++ b/ts/components/session/ActionsPanel.tsx @@ -8,6 +8,7 @@ import { UserUtils } from '../../session/utils'; import { syncConfigurationIfNeeded } from '../../session/utils/syncUtils'; import { DAYS, MINUTES } from '../../session/utils/Number'; import { + createOrUpdateItem, generateAttachmentKeyIfEmpty, getItemById, hasSyncedInitialConfigurationItem, @@ -45,6 +46,20 @@ export enum SectionType { Moon, } +const showUnstableAttachmentsDialogIfNeeded = async () => { + const alreadyShown = (await getItemById('showUnstableAttachmentsDialog'))?.value; + + if (!alreadyShown) { + window.confirmationDialog({ + title: 'File server update', + message: + "We're upgrading the way files are stored. File transfer may be unstable for the next 24-48 hours.", + }); + + await createOrUpdateItem({ id: 'showUnstableAttachmentsDialog', value: true }); + } +}; + const Section = (props: { type: SectionType; avatarPath?: string }) => { const ourNumber = useSelector(getOurNumber); const unreadMessageCount = useSelector(getUnreadMessageCount); @@ -158,6 +173,7 @@ const doAppStartUp = (dispatch: Dispatch) => { void OnionPaths.getInstance().buildNewOnionPaths(); } + void showUnstableAttachmentsDialogIfNeeded(); // init the messageQueue. In the constructor, we add all not send messages // this call does nothing except calling the constructor, which will continue sending message in the pipeline void getMessageQueue().processAllPending(); diff --git a/ts/fileserver/FileServerApiV2.ts b/ts/fileserver/FileServerApiV2.ts index 0cbf973c6..cc2262d6e 100644 --- a/ts/fileserver/FileServerApiV2.ts +++ b/ts/fileserver/FileServerApiV2.ts @@ -4,9 +4,14 @@ import { parseStatusCodeFromOnionRequest } from '../opengroup/opengroupV2/OpenGr import { fromArrayBufferToBase64, fromBase64ToArrayBuffer } from '../session/utils/String'; // tslint:disable-next-line: no-http-string -export const fileServerV2URL = 'http://88.99.175.227'; -export const fileServerV2PubKey = +export const oldFileServerV2URL = 'http://88.99.175.227'; +export const oldFileServerV2PubKey = '7cb31905b55cd5580c686911debf672577b3fb0bff81df4ce2d5c4cb3a7aaa69'; +// tslint:disable-next-line: no-http-string +export const fileServerV2URL = 'http://filev2.getsession.org'; + +export const fileServerV2PubKey = + 'da21e1d886c6fbaea313f75298bd64aab03a97ce985b46bb2dad9f2089c8ee59'; export type FileServerV2Request = { method: 'GET' | 'POST' | 'DELETE' | 'PUT'; @@ -14,6 +19,7 @@ export type FileServerV2Request = { // queryParams are used for post or get, but not the same way queryParams?: Record; headers?: Record; + isOldV2server?: boolean; // to remove in a few days }; const FILES_ENDPOINT = 'files'; @@ -67,7 +73,8 @@ export const uploadFileToFsV2 = async ( * @returns the data as an Uint8Array or null */ export const downloadFileFromFSv2 = async ( - fileIdOrCompleteUrl: string + fileIdOrCompleteUrl: string, + isOldV2server: boolean ): Promise => { let fileId = fileIdOrCompleteUrl; if (!fileIdOrCompleteUrl) { @@ -75,13 +82,19 @@ export const downloadFileFromFSv2 = async ( return null; } - const completeUrlPrefix = `${fileServerV2URL}/${FILES_ENDPOINT}/`; - if (fileIdOrCompleteUrl.startsWith(completeUrlPrefix)) { - fileId = fileId.substr(completeUrlPrefix.length); + const oldCompleteUrlPrefix = `${oldFileServerV2URL}/${FILES_ENDPOINT}/`; + const newCompleteUrlPrefix = `${fileServerV2URL}/${FILES_ENDPOINT}/`; + + if (fileIdOrCompleteUrl.startsWith(newCompleteUrlPrefix)) { + fileId = fileId.substr(newCompleteUrlPrefix.length); + } else if (fileIdOrCompleteUrl.startsWith(oldCompleteUrlPrefix)) { + fileId = fileId.substr(oldCompleteUrlPrefix.length); } + const request: FileServerV2Request = { method: 'GET', endpoint: `${FILES_ENDPOINT}/${fileId}`, + isOldV2server, }; const result = await sendApiV2Request(request); @@ -119,7 +132,11 @@ export const buildUrl = (request: FileServerV2Request | OpenGroupV2Request): URL if (isOpenGroupV2Request(request)) { rawURL = `${request.server}/${request.endpoint}`; } else { - rawURL = `${fileServerV2URL}/${request.endpoint}`; + if (request.isOldV2server) { + rawURL = `${oldFileServerV2URL}/${request.endpoint}`; + } else { + rawURL = `${fileServerV2URL}/${request.endpoint}`; + } } if (request.method === 'GET') { diff --git a/ts/opengroup/opengroupV2/OpenGroupAPIV2.ts b/ts/opengroup/opengroupV2/OpenGroupAPIV2.ts index abe7a5b48..41a649adc 100644 --- a/ts/opengroup/opengroupV2/OpenGroupAPIV2.ts +++ b/ts/opengroup/opengroupV2/OpenGroupAPIV2.ts @@ -57,7 +57,7 @@ const getDestinationPubKey = async ( } } else { // this is a fileServer call - return FSv2.fileServerV2PubKey; + return request.isOldV2server ? FSv2.oldFileServerV2PubKey : FSv2.fileServerV2PubKey; } }; diff --git a/ts/receiver/attachments.ts b/ts/receiver/attachments.ts index 36e4a1988..0cda9eb5e 100644 --- a/ts/receiver/attachments.ts +++ b/ts/receiver/attachments.ts @@ -23,18 +23,19 @@ export async function downloadAttachment(attachment: any) { serverUrl ); // is it an attachment hosted on the file server v2 ? + const defaultFsOldV2 = _.startsWith(serverUrl, FSv2.oldFileServerV2URL); const defaultFsV2 = _.startsWith(serverUrl, FSv2.fileServerV2URL); let res: ArrayBuffer | null = null; - if (defaultFsV2) { + if (defaultFsV2 || defaultFsOldV2) { let attachmentId = attachment.id; if (!attachmentId) { // try to get the fileId from the end of the URL attachmentId = attachment.url; } window.log.info('Download v2 file server attachment'); - res = await FSv2.downloadFileFromFSv2(attachmentId); + res = await FSv2.downloadFileFromFSv2(attachmentId, defaultFsOldV2); } else if (!defaultFileserver) { // TODO: we need attachments to remember which API should be used to retrieve them const serverAPI = await window.lokiPublicChatAPI.findOrCreateServer(serverUrl); From 81f8870f41afa291b34b861ba30d7db279599951 Mon Sep 17 00:00:00 2001 From: Audric Ackermann Date: Thu, 20 May 2021 13:05:13 +1000 Subject: [PATCH 02/11] Session V1.6.3 (#1647) --- ts/components/session/ActionsPanel.tsx | 16 +++++++++++ ts/fileserver/FileServerApiV2.ts | 31 +++++++++++++++++----- ts/opengroup/opengroupV2/OpenGroupAPIV2.ts | 2 +- ts/receiver/attachments.ts | 5 ++-- 4 files changed, 44 insertions(+), 10 deletions(-) diff --git a/ts/components/session/ActionsPanel.tsx b/ts/components/session/ActionsPanel.tsx index 5ced06163..f7cb86005 100644 --- a/ts/components/session/ActionsPanel.tsx +++ b/ts/components/session/ActionsPanel.tsx @@ -8,6 +8,7 @@ import { UserUtils } from '../../session/utils'; import { syncConfigurationIfNeeded } from '../../session/utils/syncUtils'; import { DAYS, MINUTES } from '../../session/utils/Number'; import { + createOrUpdateItem, generateAttachmentKeyIfEmpty, getItemById, hasSyncedInitialConfigurationItem, @@ -45,6 +46,20 @@ export enum SectionType { Moon, } +const showUnstableAttachmentsDialogIfNeeded = async () => { + const alreadyShown = (await getItemById('showUnstableAttachmentsDialog'))?.value; + + if (!alreadyShown) { + window.confirmationDialog({ + title: 'File server update', + message: + "We're upgrading the way files are stored. File transfer may be unstable for the next 24-48 hours.", + }); + + await createOrUpdateItem({ id: 'showUnstableAttachmentsDialog', value: true }); + } +}; + const Section = (props: { type: SectionType; avatarPath?: string }) => { const ourNumber = useSelector(getOurNumber); const unreadMessageCount = useSelector(getUnreadMessageCount); @@ -158,6 +173,7 @@ const doAppStartUp = (dispatch: Dispatch) => { void OnionPaths.getInstance().buildNewOnionPaths(); } + void showUnstableAttachmentsDialogIfNeeded(); // init the messageQueue. In the constructor, we add all not send messages // this call does nothing except calling the constructor, which will continue sending message in the pipeline void getMessageQueue().processAllPending(); diff --git a/ts/fileserver/FileServerApiV2.ts b/ts/fileserver/FileServerApiV2.ts index 0cbf973c6..cc2262d6e 100644 --- a/ts/fileserver/FileServerApiV2.ts +++ b/ts/fileserver/FileServerApiV2.ts @@ -4,9 +4,14 @@ import { parseStatusCodeFromOnionRequest } from '../opengroup/opengroupV2/OpenGr import { fromArrayBufferToBase64, fromBase64ToArrayBuffer } from '../session/utils/String'; // tslint:disable-next-line: no-http-string -export const fileServerV2URL = 'http://88.99.175.227'; -export const fileServerV2PubKey = +export const oldFileServerV2URL = 'http://88.99.175.227'; +export const oldFileServerV2PubKey = '7cb31905b55cd5580c686911debf672577b3fb0bff81df4ce2d5c4cb3a7aaa69'; +// tslint:disable-next-line: no-http-string +export const fileServerV2URL = 'http://filev2.getsession.org'; + +export const fileServerV2PubKey = + 'da21e1d886c6fbaea313f75298bd64aab03a97ce985b46bb2dad9f2089c8ee59'; export type FileServerV2Request = { method: 'GET' | 'POST' | 'DELETE' | 'PUT'; @@ -14,6 +19,7 @@ export type FileServerV2Request = { // queryParams are used for post or get, but not the same way queryParams?: Record; headers?: Record; + isOldV2server?: boolean; // to remove in a few days }; const FILES_ENDPOINT = 'files'; @@ -67,7 +73,8 @@ export const uploadFileToFsV2 = async ( * @returns the data as an Uint8Array or null */ export const downloadFileFromFSv2 = async ( - fileIdOrCompleteUrl: string + fileIdOrCompleteUrl: string, + isOldV2server: boolean ): Promise => { let fileId = fileIdOrCompleteUrl; if (!fileIdOrCompleteUrl) { @@ -75,13 +82,19 @@ export const downloadFileFromFSv2 = async ( return null; } - const completeUrlPrefix = `${fileServerV2URL}/${FILES_ENDPOINT}/`; - if (fileIdOrCompleteUrl.startsWith(completeUrlPrefix)) { - fileId = fileId.substr(completeUrlPrefix.length); + const oldCompleteUrlPrefix = `${oldFileServerV2URL}/${FILES_ENDPOINT}/`; + const newCompleteUrlPrefix = `${fileServerV2URL}/${FILES_ENDPOINT}/`; + + if (fileIdOrCompleteUrl.startsWith(newCompleteUrlPrefix)) { + fileId = fileId.substr(newCompleteUrlPrefix.length); + } else if (fileIdOrCompleteUrl.startsWith(oldCompleteUrlPrefix)) { + fileId = fileId.substr(oldCompleteUrlPrefix.length); } + const request: FileServerV2Request = { method: 'GET', endpoint: `${FILES_ENDPOINT}/${fileId}`, + isOldV2server, }; const result = await sendApiV2Request(request); @@ -119,7 +132,11 @@ export const buildUrl = (request: FileServerV2Request | OpenGroupV2Request): URL if (isOpenGroupV2Request(request)) { rawURL = `${request.server}/${request.endpoint}`; } else { - rawURL = `${fileServerV2URL}/${request.endpoint}`; + if (request.isOldV2server) { + rawURL = `${oldFileServerV2URL}/${request.endpoint}`; + } else { + rawURL = `${fileServerV2URL}/${request.endpoint}`; + } } if (request.method === 'GET') { diff --git a/ts/opengroup/opengroupV2/OpenGroupAPIV2.ts b/ts/opengroup/opengroupV2/OpenGroupAPIV2.ts index abe7a5b48..41a649adc 100644 --- a/ts/opengroup/opengroupV2/OpenGroupAPIV2.ts +++ b/ts/opengroup/opengroupV2/OpenGroupAPIV2.ts @@ -57,7 +57,7 @@ const getDestinationPubKey = async ( } } else { // this is a fileServer call - return FSv2.fileServerV2PubKey; + return request.isOldV2server ? FSv2.oldFileServerV2PubKey : FSv2.fileServerV2PubKey; } }; diff --git a/ts/receiver/attachments.ts b/ts/receiver/attachments.ts index 36e4a1988..0cda9eb5e 100644 --- a/ts/receiver/attachments.ts +++ b/ts/receiver/attachments.ts @@ -23,18 +23,19 @@ export async function downloadAttachment(attachment: any) { serverUrl ); // is it an attachment hosted on the file server v2 ? + const defaultFsOldV2 = _.startsWith(serverUrl, FSv2.oldFileServerV2URL); const defaultFsV2 = _.startsWith(serverUrl, FSv2.fileServerV2URL); let res: ArrayBuffer | null = null; - if (defaultFsV2) { + if (defaultFsV2 || defaultFsOldV2) { let attachmentId = attachment.id; if (!attachmentId) { // try to get the fileId from the end of the URL attachmentId = attachment.url; } window.log.info('Download v2 file server attachment'); - res = await FSv2.downloadFileFromFSv2(attachmentId); + res = await FSv2.downloadFileFromFSv2(attachmentId, defaultFsOldV2); } else if (!defaultFileserver) { // TODO: we need attachments to remember which API should be used to retrieve them const serverAPI = await window.lokiPublicChatAPI.findOrCreateServer(serverUrl); From ab22126b45d490afc23040fb46be2b7483e48c34 Mon Sep 17 00:00:00 2001 From: Audric Ackermann Date: Thu, 20 May 2021 14:12:06 +1000 Subject: [PATCH 03/11] bump to 1.6.3 (#1648) * switch to dedicated server * bump to 1.6.3 --- package.json | 2 +- 1 file changed, 1 insertion(+), 1 deletion(-) diff --git a/package.json b/package.json index cdf95c932..52d882c81 100644 --- a/package.json +++ b/package.json @@ -2,7 +2,7 @@ "name": "session-desktop", "productName": "Session", "description": "Private messaging from your desktop", - "version": "1.6.2", + "version": "1.6.3", "license": "GPL-3.0", "author": { "name": "Loki Project", From 76256383a96c4cecbc2e4d36acc39819d9ce0d32 Mon Sep 17 00:00:00 2001 From: Audric Ackermann Date: Thu, 20 May 2021 14:12:41 +1000 Subject: [PATCH 04/11] bump to 1.6.3 (#1649) --- package.json | 2 +- 1 file changed, 1 insertion(+), 1 deletion(-) diff --git a/package.json b/package.json index cdf95c932..52d882c81 100644 --- a/package.json +++ b/package.json @@ -2,7 +2,7 @@ "name": "session-desktop", "productName": "Session", "description": "Private messaging from your desktop", - "version": "1.6.2", + "version": "1.6.3", "license": "GPL-3.0", "author": { "name": "Loki Project", From b06f7320153612c00bfed4dcca36e39ec0afc012 Mon Sep 17 00:00:00 2001 From: Audric Ackermann Date: Mon, 24 May 2021 13:20:20 +1000 Subject: [PATCH 05/11] add instructions to verify signatures in readme (#1651) --- README.md | 33 +++++++++++++++++++++++++++++++++ 1 file changed, 33 insertions(+) diff --git a/README.md b/README.md index d16e9c083..1855f8500 100644 --- a/README.md +++ b/README.md @@ -14,6 +14,39 @@ Please search for any [existing issues](https://github.com/oxen-io/session-deskt Build instructions can be found in [BUILDING.md](BUILDING.md). + +## Verifing signatures + + +Get Kee's key and import it: +``` +wget https://raw.githubusercontent.com/oxen-io/oxen-core/master/utils/gpg_keys/KeeJef.asc +gpg --import KeeJef.asc +``` + +Get the signed hash for this release, the SESSION_VERSION needs to be updated for the release you want to verify +``` +export SESSION_VERSION=1.6.1 +wget https://github.com/oxen-io/session-desktop/releases/download/v$SESSION_VERSION/signatures.asc +``` + +Verify the signature of the hashes of the files + +``` +gpg --verify signatures.asc 2>&1 |grep "Good signature from" +``` + +The command above should print "`Good signature from "Kee Jefferys...`" +If it does, the hashes are valid but we still have to make the sure the signed hashes matches the downloaded files. + +Make sure the two commands below returns the same hash. +If they do, files are valid +``` +sha256sum session-desktop-linux-amd64-$SESSION_VERSION.deb +grep .deb signatures.asc +``` + + ## Debian repository Please visit https://deb.oxen.io/
From 4f55bb42812a120ef0ed11ee75ad89cb32e39113 Mon Sep 17 00:00:00 2001 From: Audric Ackermann Date: Mon, 24 May 2021 13:31:26 +1000 Subject: [PATCH 06/11] remove unstable attachment for next 48 hours dialog (#1653) --- ts/components/session/ActionsPanel.tsx | 17 ----------------- 1 file changed, 17 deletions(-) diff --git a/ts/components/session/ActionsPanel.tsx b/ts/components/session/ActionsPanel.tsx index f7cb86005..b6c6c90e1 100644 --- a/ts/components/session/ActionsPanel.tsx +++ b/ts/components/session/ActionsPanel.tsx @@ -8,11 +8,9 @@ import { UserUtils } from '../../session/utils'; import { syncConfigurationIfNeeded } from '../../session/utils/syncUtils'; import { DAYS, MINUTES } from '../../session/utils/Number'; import { - createOrUpdateItem, generateAttachmentKeyIfEmpty, getItemById, hasSyncedInitialConfigurationItem, - removeItemById, } from '../../data/data'; import { OnionPaths } from '../../session/onions'; import { getMessageQueue } from '../../session/sending'; @@ -46,20 +44,6 @@ export enum SectionType { Moon, } -const showUnstableAttachmentsDialogIfNeeded = async () => { - const alreadyShown = (await getItemById('showUnstableAttachmentsDialog'))?.value; - - if (!alreadyShown) { - window.confirmationDialog({ - title: 'File server update', - message: - "We're upgrading the way files are stored. File transfer may be unstable for the next 24-48 hours.", - }); - - await createOrUpdateItem({ id: 'showUnstableAttachmentsDialog', value: true }); - } -}; - const Section = (props: { type: SectionType; avatarPath?: string }) => { const ourNumber = useSelector(getOurNumber); const unreadMessageCount = useSelector(getUnreadMessageCount); @@ -173,7 +157,6 @@ const doAppStartUp = (dispatch: Dispatch) => { void OnionPaths.getInstance().buildNewOnionPaths(); } - void showUnstableAttachmentsDialogIfNeeded(); // init the messageQueue. In the constructor, we add all not send messages // this call does nothing except calling the constructor, which will continue sending message in the pipeline void getMessageQueue().processAllPending(); From 27c881533577b01edae0692bf728d64e75569bac Mon Sep 17 00:00:00 2001 From: Audric Ackermann Date: Mon, 24 May 2021 13:38:11 +1000 Subject: [PATCH 07/11] Remove unstable attachment dialog (#1655) --- package.json | 2 +- 1 file changed, 1 insertion(+), 1 deletion(-) diff --git a/package.json b/package.json index 52d882c81..960e5f7fd 100644 --- a/package.json +++ b/package.json @@ -2,7 +2,7 @@ "name": "session-desktop", "productName": "Session", "description": "Private messaging from your desktop", - "version": "1.6.3", + "version": "1.6.4", "license": "GPL-3.0", "author": { "name": "Loki Project", From d86e8cd1981edc322450ffe19bb130c60b3694b5 Mon Sep 17 00:00:00 2001 From: Warrick Date: Mon, 24 May 2021 15:48:20 +1000 Subject: [PATCH 08/11] Does not display pubkey on message author unless its an open group chat. (#1656) --- ts/components/ConversationListItem.tsx | 2 +- 1 file changed, 1 insertion(+), 1 deletion(-) diff --git a/ts/components/ConversationListItem.tsx b/ts/components/ConversationListItem.tsx index a2404c912..76f1ce05f 100644 --- a/ts/components/ConversationListItem.tsx +++ b/ts/components/ConversationListItem.tsx @@ -229,7 +229,7 @@ class ConversationListItem extends React.PureComponent { const displayName = isMe ? i18n('noteToSelf') : profileName; let shouldShowPubkey = false; - if (!name || name.length === 0) { + if ((!name || name.length === 0) && (!displayName || displayName.length === 0)) { shouldShowPubkey = true; } From 1f99f2f71ceb0a2dfd6687336cdfd0038e750663 Mon Sep 17 00:00:00 2001 From: Audric Ackermann Date: Tue, 25 May 2021 10:22:26 +1000 Subject: [PATCH 09/11] use attachment name when we have one for file saving --- ts/test/types/Attachment_test.ts | 32 ++++++++++++++++++++++++-------- ts/types/Attachment.ts | 3 +++ 2 files changed, 27 insertions(+), 8 deletions(-) diff --git a/ts/test/types/Attachment_test.ts b/ts/test/types/Attachment_test.ts index 55bf2dd56..a30fc03bc 100644 --- a/ts/test/types/Attachment_test.ts +++ b/ts/test/types/Attachment_test.ts @@ -47,7 +47,7 @@ describe('Attachment', () => { contentType: MIME.VIDEO_QUICKTIME, }; const actual = Attachment.getSuggestedFilename({ attachment }); - const expected = 'session-attachment.mov'; + const expected = 'funny-cat.mov'; assert.strictEqual(actual, expected); }); it('should generate a filename without timestamp but with an index', () => { @@ -60,7 +60,7 @@ describe('Attachment', () => { attachment, index: 3, }); - const expected = 'session-attachment_003.mov'; + const expected = 'funny-cat.mov'; assert.strictEqual(actual, expected); }); it('should generate a filename with an extension if contentType is not setup', () => { @@ -73,7 +73,7 @@ describe('Attachment', () => { attachment, index: 3, }); - const expected = 'session-attachment_003.ini'; + const expected = 'funny-cat.ini'; assert.strictEqual(actual, expected); }); @@ -87,7 +87,7 @@ describe('Attachment', () => { attachment, index: 3, }); - const expected = 'session-attachment_003.txt'; + const expected = 'funny-cat.txt'; assert.strictEqual(actual, expected); }); it('should generate a filename with an extension if contentType is json', () => { @@ -100,7 +100,7 @@ describe('Attachment', () => { attachment, index: 3, }); - const expected = 'session-attachment_003.json'; + const expected = 'funny-cat.json'; assert.strictEqual(actual, expected); }); }); @@ -116,14 +116,14 @@ describe('Attachment', () => { attachment, timestamp, }); - const expected = 'session-attachment-2000-01-01-000000.mov'; + const expected = 'funny-cat.mov'; assert.strictEqual(actual, expected); }); }); context('for attachment with index', () => { - it('should generate a filename based on timestamp', () => { + it('should generate a filename based on timestamp if filename is not set', () => { const attachment: Attachment.AttachmentType = { - fileName: 'funny-cat.mov', + fileName: '', url: 'funny-cat.mov', contentType: MIME.VIDEO_QUICKTIME, }; @@ -136,6 +136,22 @@ describe('Attachment', () => { const expected = 'session-attachment-1970-01-01-000000_003.mov'; assert.strictEqual(actual, expected); }); + + it('should generate a filename based on filename if present', () => { + const attachment: Attachment.AttachmentType = { + fileName: 'funny-cat.mov', + url: 'funny-cat.mov', + contentType: MIME.VIDEO_QUICKTIME, + }; + const timestamp = new Date(new Date(0).getTimezoneOffset() * 60 * 1000); + const actual = Attachment.getSuggestedFilename({ + attachment, + timestamp, + index: 3, + }); + const expected = 'funny-cat.mov'; + assert.strictEqual(actual, expected); + }); }); }); diff --git a/ts/types/Attachment.ts b/ts/types/Attachment.ts index b162c267a..cdb632a37 100644 --- a/ts/types/Attachment.ts +++ b/ts/types/Attachment.ts @@ -334,6 +334,9 @@ export const getSuggestedFilename = ({ timestamp?: number | Date; index?: number; }): string => { + if (attachment.fileName?.length > 3) { + return attachment.fileName; + } const prefix = 'session-attachment'; const suffix = timestamp ? moment(timestamp).format('-YYYY-MM-DD-HHmmss') : ''; const fileType = getFileExtension(attachment); From 8c726fd7451d0b0fa516e0df65cedafe8051a4b0 Mon Sep 17 00:00:00 2001 From: Audric Ackermann Date: Tue, 25 May 2021 13:19:34 +1000 Subject: [PATCH 10/11] reupload avatar every 10 days --- background.html | 4 +- js/background.js | 22 ++---- libtextsecure/crypto.d.ts | 1 + preload.js | 1 + ts/components/Avatar.tsx | 6 +- ts/components/session/ActionsPanel.tsx | 96 +++++++++++++++++++++++++- ts/data/data.ts | 1 + ts/models/conversation.ts | 22 +----- 8 files changed, 112 insertions(+), 41 deletions(-) diff --git a/background.html b/background.html index 7d02dfdb6..58cf594e8 100644 --- a/background.html +++ b/background.html @@ -10,7 +10,7 @@
diff --git a/ts/data/data.ts b/ts/data/data.ts index d87f130b8..c5c4fb5d7 100644 --- a/ts/data/data.ts +++ b/ts/data/data.ts @@ -60,6 +60,7 @@ export type ServerToken = { }; export const hasSyncedInitialConfigurationItem = 'hasSyncedInitialConfigurationItem'; +export const lastAvatarUploadTimestamp = 'lastAvatarUploadTimestamp'; const channelsToMake = { shutdown, diff --git a/ts/models/conversation.ts b/ts/models/conversation.ts index aa92d145a..f347cc5c3 100644 --- a/ts/models/conversation.ts +++ b/ts/models/conversation.ts @@ -179,7 +179,6 @@ export class ConversationModel extends Backbone.Model { //start right away the function is called, and wait 1sec before calling it again this.markRead = _.debounce(this.markReadBouncy, 1000, { leading: true }); // Listening for out-of-band data updates - this.on('ourAvatarChanged', avatar => this.updateAvatarOnPublicChat(avatar)); this.typingRefreshTimer = null; this.typingPauseTimer = null; @@ -778,23 +777,6 @@ export class ConversationModel extends Backbone.Model { return null; } - public async updateAvatarOnPublicChat({ url, profileKey }: any) { - if (!this.isPublic()) { - return; - } - // Always share avatars on PublicChat - - if (profileKey && typeof profileKey !== 'string') { - // eslint-disable-next-line no-param-reassign - // tslint:disable-next-line: no-parameter-reassignment - profileKey = fromArrayBufferToBase64(profileKey); - } - const serverAPI = await window.lokiPublicChatAPI.findOrCreateServer(this.get('server')); - if (!serverAPI) { - return; - } - await serverAPI.setAvatar(url, profileKey); - } public async bouncyUpdateLastMessage() { if (!this.id) { return; @@ -1223,11 +1205,11 @@ export class ConversationModel extends Backbone.Model { // Not sure if we care about updating the database } - public async setProfileAvatar(avatar: any, avatarHash?: string) { + public async setProfileAvatar(avatar: null | { path: string }, avatarHash?: string) { const profileAvatar = this.get('avatar'); const existingHash = this.get('avatarHash'); let shouldCommit = false; - if (profileAvatar !== avatar) { + if (!_.isEqual(profileAvatar, avatar)) { this.set({ avatar }); shouldCommit = true; } From 1afbd2809a3dfb29c798d89965451513eff0b765 Mon Sep 17 00:00:00 2001 From: Audric Ackermann Date: Tue, 25 May 2021 13:23:08 +1000 Subject: [PATCH 11/11] lint --- ts/components/session/ActionsPanel.tsx | 7 ++----- 1 file changed, 2 insertions(+), 5 deletions(-) diff --git a/ts/components/session/ActionsPanel.tsx b/ts/components/session/ActionsPanel.tsx index f1afbc886..129da2a27 100644 --- a/ts/components/session/ActionsPanel.tsx +++ b/ts/components/session/ActionsPanel.tsx @@ -7,7 +7,6 @@ import { ConversationController } from '../../session/conversations'; import { UserUtils } from '../../session/utils'; import { syncConfigurationIfNeeded } from '../../session/utils/syncUtils'; import { DAYS, MINUTES } from '../../session/utils/Number'; -import fse from 'fs-extra'; import { createOrUpdateItem, @@ -176,12 +175,10 @@ const triggerSyncIfNeeded = async () => { const triggerAvatarReUploadIfNeeded = async () => { const lastTimeStampAvatarUpload = (await getItemById(lastAvatarUploadTimestamp))?.value || 0; - if (Date.now() - lastTimeStampAvatarUpload > 14 * DAYS) { + if (Date.now() - lastTimeStampAvatarUpload > DAYS * 14) { window.log.info('Reuploading avatar...'); // reupload the avatar - const ourConvo = await ConversationController.getInstance().get( - UserUtils.getOurPubKeyStrFromCache() - ); + const ourConvo = ConversationController.getInstance().get(UserUtils.getOurPubKeyStrFromCache()); if (!ourConvo) { window.log.warn('ourConvo not found... This is not a valid case'); return;