Обещание приоритета при попытке загрузить файл в Google Storage - PullRequest
Новая
       10

Обещание приоритета при попытке загрузить файл в Google Storage

0 голосов
/ 07 июня 2018

Я загружаю файл в Google Cloud.По завершении я хочу вернуть URL файла и некоторые метаданные.Я пытаюсь сделать это так: 

blobStream.on('finish', () => {

                var fileSize;
                var fileCreated;

                fileUpload.getMetadata()
                    .then(metadata => { 
                        const mObj = metadata[0];
                        fileSize = mObj.size;
                        fileCreated = mObj.timeCreated;
                });

                fileUpload.getSignedUrl({
                    action: 'read',
                    expires: '03-09-2491'
                }).then(signedUrl => {
                    let response = {};
                        response.name = newFileName;
                        response.size = fileSize;
                        response.created = fileCreated;
                        response.url = signedUrl[0];
                    resolve(response);
                }).catch(error => {
                    reject(error);  
                });

            });

Проблема в том, что метод getSignedUrl всегда срабатывает первым, а fileSize + fileCreated никогда не заполняется.

Я считаю, что мне нужно объединить события, чтобы сначала произошла getMetadata, но я относительно новичок в Node (и в Promises) и мог бы использовать некоторые предложения.Ниже работает метод) 

uploadFileToStorage(file) {

    let utils = new Utils();
    let newFileName = `${utils._guid()}-${Date.now()}`;
    let bucket = admin.storage().bucket();
    let filePath = url.format('audioFiles/' + newFileName);
    let fileUpload = bucket.file(filePath);

    let prom = new Promise(function(resolve, reject) {

        if (!file) {
            reject('No file provided');
            return;
        }

        const blobStream = fileUpload.createWriteStream({
            metadata: {
                contentType: file.mimetype
            }
        });

        blobStream.on('error', (error) => {
            reject('Something is wrong! Unable to upload at the moment.');
            return;
         });

        blobStream.on('finish', resolve);

        blobStream.end(file.buffer);

      }).then(() => { 

        return Promise.all([
            fileUpload.getMetadata(),
            fileUpload.getSignedUrl({
                action: 'read',
                expires: '03-09-2491'
            }),
        ]);

    }).then(([metadata, signedUrl]) => {

        const fileSize = metadata[0].size;
        const fileCreated = metadata[0].timeCreated;
        const response = {
            name: newFileName,
            size: fileSize,
            created: fileCreated,
            url: signedUrl[0],
        };

        return response;

    });

    return prom;

}

1 Ответ

0 голосов
/ 08 июня 2018

Не создавайте двух независимых обещаний и не используйте переменные внешней области действия, которые вы никогда не знаете, когда они будут назначены.Вместо этого используйте Promise.all для ожидания нескольких обещаний: 

new Promise(resolve => {
    blobStream.on('finish', resolve);
}).then(() => {
    return Promise.all([
        fileUpload.getMetadata(),
        fileUpload.getSignedUrl({
            action: 'read',
            expires: '03-09-2491'
        }),
    ]);
}).then(([metadata, signedUrl]) => {
    const fileSize = metadata[0].size;
    const fileCreated = metadata[0].timeCreated;
    const response = {
        name: newFileName,
        size: fileSize,
        created: fileCreated,
        url: signedUrl[0],
    });
    return response;
});

Также избегайте , используя дополнительные обещания внутри конструктора Promise !

...