Attachments on Android

mojoe · 10 September 2022 14:09

The online help still says that attachments over 10 MB will crash the mobile app when syncing. I'm assuming that this is no longer true?

I just imported everything from Evernote into Joplin Desktop, setup encryption, then synced to my Android phone via Dropbox. I have many attachments larger than 10 MB. I was able to open several of them, including the largest (152 MB) on the phone.

MrKanister · 11 September 2022 17:59

Essentially the issue still persists: The mobile app can display resources greater than 10 MB without crashing but not sync them.

This was the issue: Mobile - App crashes when syncing large resources (greated than 10MB) · Issue #371 · laurent22/joplin · GitHub

And this is the commit All: Fixes #371 (sort of): Allow resources greater than 10 MB but the… · laurent22/joplin@565dfba · GitHub

roman_r_m · 11 September 2022 19:26

The limit is 100mb, not 10. Even that isn't needed in my opinion

MrKanister · 11 September 2022 20:43

So resources ≤100 MB get synced? Why is it no longer needed?

roman_r_m · 11 September 2022 20:57

Yes:

github.com

laurent22/joplin/blob/f59d29f1c56442bcd1669f6c1dcd6500cde8a1fb/packages/lib/Synchronizer.ts#L138


      
          	}
          
          	migrationHandler() {
          		if (this.migrationHandler_) return this.migrationHandler_;
          		this.migrationHandler_ = new MigrationHandler(this.api(), this.db(), this.lockHandler(), this.lockClientType(), this.clientId_);
          		return this.migrationHandler_;
          	}
          
          	maxResourceSize() {
          		if (this.maxResourceSize_ !== null) return this.maxResourceSize_;
          		return this.appType_ === AppType.Mobile ? 100 * 1000 * 1000 : Infinity;
          	}
          
          	public setShareService(v: ShareService) {
          		this.shareService_ = v;
          	}
          
          	public setEncryptionService(v: any) {
          		this.encryptionService_ = v;
          	}

The underlying bug has been fixed long ago, the limit is an extra precaution.

MrKanister · 11 September 2022 21:21

I see. Thanks for clarifying.

mojoe · 11 September 2022 21:48

So, that 152 MB attachment that synced to my phone, and that I opened, must have been an illusion or a fluke.

roman_r_m · 12 September 2022 03:26

Yeah, that's interesting. Could be a bug in this check I suppose

wh201906 · 12 September 2022 04:19

How about this threshold?

github.com

laurent22/joplin/blob/f59d29f1c56442bcd1669f6c1dcd6500cde8a1fb/packages/app-mobile/components/screens/Note.tsx#L869


      
          				const done = await this.resizeImage(localFilePath, targetPath, mimeType);
          				if (!done) return;
          			} else {
          				if (fileType === 'image') {
          					dialogs.error(this, _('Unsupported image type: %s', mimeType));
          					return;
          				} else {
          					await shim.fsDriver().copy(localFilePath, targetPath);
          
          
					const stat = await shim.fsDriver().stat(targetPath);
          					if (stat.size >= 10000000) {
          						await shim.fsDriver().remove(targetPath);
          						throw new Error('Resources larger than 10 MB are not currently supported as they may crash the mobile applications. The issue is being investigated and will be fixed at a later time.');
          					}
          				}
          			}
          		} catch (error) {
          			reg.logger().warn('Could not attach file:', error);
          			await dialogs.error(this, error.message);
          			return;
          		}

roman_r_m · 12 September 2022 05:57

Forgot about this one! I think it can be removed too, there's no need to keep it.

However this limit only applies when attaching files on mobile. The original question was about a file attached elsewhere and synced to mobile.

wh201906 · 12 September 2022 06:01

Ok

system · 12 October 2022 06:01

This topic was automatically closed 30 days after the last reply. New replies are no longer allowed.

Topic		Replies	Views
What's the latest on resource sizes / limits? Support	33	4647	22 June 2021
[Android] App crashes while syncing and decrypting Support	9	1026	23 July 2019
How do attachments work? Support	2	571	26 December 2019
What's the file size limit for an attachment in Android? #2146 Support	17	2869	21 April 2020
Android synch problem where attachments are too large Support	8	321	10 May 2024

Attachments on Android

Related topics