Export: Limit path lengths of attachments, add tests #1439
Merged
Add this suggestion to a batch that can be applied as a single commit.
This suggestion is invalid because no changes were made to the code.
Suggestions cannot be applied while the pull request is closed.
Suggestions cannot be applied while viewing a subset of changes.
Only one suggestion per line can be applied in a batch.
Add this suggestion to a batch that can be applied as a single commit.
Applying suggestions on deleted lines is not supported.
You must change the existing code in this line in order to create a valid suggestion.
Outdated suggestions cannot be applied.
This suggestion has been applied or marked resolved.
Suggestions cannot be applied from pending reviews.
Suggestions cannot be applied on multi-line comments.
Suggestions cannot be applied while the pull request is queued to merge.
Suggestion cannot be applied right now. Please check back later.
Three changes:
backup.js
helper functionsThis should make it more likely that we'll succeed on Windows, which has a path length limit.
Bonus: Fix
MessageView
tests, which were failing during code coverage runsNote: This is an export format breaking change, since it changes how attachment filenames are generated. Any attachment filename longer than 30 characters will cause a failure on import if using an older build that doesn't trim filenames.