rmorawski/standardnotes-app-web
1
0
Fork 0
Code Issues Pull Requests Actions Packages Projects Releases Wiki Activity

fix: sanitize backup file names for windows

Browse Source
This commit is contained in:
Baptiste Grob
2021-03-03 16:46:31 +01:00
parent a14b7ed411
commit 9d017b5745
1 changed files with 27 additions and 24 deletions
Download Patch File Download Diff File Expand all files Collapse all files

43
app/assets/javascripts/services/archiveManager.ts
View File

@@ -7,21 +7,25 @@ import {
PayloadContent, PayloadContent,
} from '@standardnotes/snjs'; } from '@standardnotes/snjs';
function zippableTxtName(name: string, suffix = ""): string { function sanitizeFileName(name: string) {
const sanitizedName = name return name
.replace(/\//g, '') .replace(/\//g, '')
.replace(/\\+/g, '') .replace(/\\+/g, '')
.replace(/:/g, ' ') .replace(/:/g, ' ')
.replace(/\|/g, ' ')
.replace(/\./g, ' '); .replace(/\./g, ' ');
const nameEnd = suffix + ".txt"; }
function zippableTxtName(name: string, suffix = ''): string {
const sanitizedName = sanitizeFileName(name);
const nameEnd = suffix + '.txt';
const maxFileNameLength = 255; const maxFileNameLength = 255;
return sanitizedName.slice(0, maxFileNameLength - nameEnd.length) + nameEnd; return sanitizedName.slice(0, maxFileNameLength - nameEnd.length) + nameEnd;
} }
export class ArchiveManager { export class ArchiveManager {
private readonly application: WebApplication;
private readonly application: WebApplication private textFile?: string;
private textFile?: string
constructor(application: WebApplication) { constructor(application: WebApplication) {
this.application = application; this.application = application;
@@ -36,10 +40,9 @@ export class ArchiveManager {
if (!data) { if (!data) {
return; return;
} }
const blobData = new Blob( const blobData = new Blob([JSON.stringify(data, null, 2)], {
[JSON.stringify(data, null, 2)], type: 'text/json',
{ type: 'text/json' } });
);
if (encrypted) { if (encrypted) {
this.downloadData( this.downloadData(
blobData, blobData,
@@ -85,21 +88,18 @@ export class ArchiveManager {
}); });
} }
private async downloadZippedDecryptedItems( private async downloadZippedDecryptedItems(data: BackupFile) {
data: BackupFile
) {
await this.loadZip(); await this.loadZip();
const items = data.items; const items = data.items;
this.zip.createWriter( this.zip.createWriter(
new this.zip.BlobWriter('application/zip'), new this.zip.BlobWriter('application/zip'),
async (zipWriter: any) => { async (zipWriter: any) => {
await new Promise((resolve) => { await new Promise((resolve) => {
const blob = new Blob( const blob = new Blob([JSON.stringify(data, null, 2)], {
[JSON.stringify(data, null, 2)], type: 'text/plain',
{ type: 'text/plain' } });
);
const fileName = zippableTxtName( const fileName = zippableTxtName(
'Standard Notes Backup and Import File.txt' 'Standard Notes Backup and Import File'
); );
zipWriter.add(fileName, new this.zip.BlobReader(blob), resolve); zipWriter.add(fileName, new this.zip.BlobReader(blob), resolve);
}); });
@@ -120,7 +120,8 @@ export class ArchiveManager {
name = ''; name = '';
} }
const blob = new Blob([contents], { type: 'text/plain' }); const blob = new Blob([contents], { type: 'text/plain' });
const fileName = `Items/${item.content_type}/` + const fileName =
`Items/${sanitizeFileName(item.content_type)}/` +
zippableTxtName(name, `-${item.uuid.split('-')[0]}`); zippableTxtName(name, `-${item.uuid.split('-')[0]}`);
zipWriter.add(fileName, new this.zip.BlobReader(blob), () => { zipWriter.add(fileName, new this.zip.BlobReader(blob), () => {
index++; index++;
@@ -138,7 +139,9 @@ export class ArchiveManager {
}); });
}; };
nextFile(); nextFile();
}, onerror); },
onerror
);
} }
private hrefForData(data: Blob) { private hrefForData(data: Blob) {