firebase-tools 10.1.1 → 10.1.2

package/lib/accountExporter.js

@@ -2,9 +2,13 @@
 var os = require("os");
 var path = require("path");
 var _ = require("lodash");
-var api = require("./api");
-var utils = require("./utils");
+const { Client } = require("./apiv2");
+const { googleOrigin } = require("./api");
 var { FirebaseError } = require("./error");
+var utils = require("./utils");
+const apiClient = new Client({
+    urlPrefix: googleOrigin,
+});
 var EXPORTED_JSON_KEYS = [
     "localId",
     "email",
@@ -149,12 +153,9 @@ var serialExportUsers = function (projectId, options) {
     if (!options.timeoutRetryCount) {
         options.timeoutRetryCount = 0;
     }
-    return api
-        .request("POST", "/identitytoolkit/v3/relyingparty/downloadAccount", {
-        auth: true,
-        json: true,
-        data: postBody,
-        origin: api.googleOrigin,
+    return apiClient
+        .post("/identitytoolkit/v3/relyingparty/downloadAccount", postBody, {
+        skipLog: { resBody: true },
     })
         .then(function (ret) {
         options.timeoutRetryCount = 0;

package/lib/accountImporter.js

@@ -1,10 +1,14 @@
 "use strict";
 var clc = require("cli-color");
 var _ = require("lodash");
-var api = require("./api");
+const { Client } = require("./apiv2");
+const { googleOrigin } = require("./api");
 const { logger } = require("./logger");
-var utils = require("./utils");
 var { FirebaseError } = require("./error");
+var utils = require("./utils");
+const apiClient = new Client({
+    urlPrefix: googleOrigin,
+});
 var ALLOWED_JSON_KEYS = [
     "localId",
     "email",
@@ -266,12 +270,10 @@ var validateUserJson = function (userJson) {
 };
 var _sendRequest = function (projectId, userList, hashOptions) {
     logger.info("Starting importing " + userList.length + " account(s).");
-    return api
-        .request("POST", "/identitytoolkit/v3/relyingparty/uploadAccount", {
-        auth: true,
-        json: true,
-        data: _genUploadAccountPostBody(projectId, userList, hashOptions),
-        origin: api.googleOrigin,
+    const postData = _genUploadAccountPostBody(projectId, userList, hashOptions);
+    return apiClient
+        .post("/identitytoolkit/v3/relyingparty/uploadAccount", postData, {
+        skipLog: { body: true },
     })
         .then(function (ret) {
         if (ret.body.error) {

package/lib/bin/firebase.js

@@ -15,7 +15,7 @@ if (!semver.satisfies(nodeVersion, pkg.engines.node)) {
 const updateNotifier = require("update-notifier")({ pkg: pkg });
 const clc = require("cli-color");
 const TerminalRenderer = require("marked-terminal");
-const marked = require("marked");
+const marked = require("marked").marked;
 marked.setOptions({
     renderer: new TerminalRenderer(),
 });

package/lib/commands/auth-export.js

@@ -1,39 +1,40 @@
 "use strict";
-var clc = require("cli-color");
-var fs = require("fs");
-var os = require("os");
-var { Command } = require("../command");
-var accountExporter = require("../accountExporter");
-var needProjectId = require("../projectUtils").needProjectId;
-const { logger } = require("../logger");
-var { requirePermissions } = require("../requirePermissions");
-var MAX_BATCH_SIZE = 1000;
-var validateOptions = accountExporter.validateOptions;
-var serialExportUsers = accountExporter.serialExportUsers;
-module.exports = new Command("auth:export [dataFile]")
+Object.defineProperty(exports, "__esModule", { value: true });
+const clc = require("cli-color");
+const fs = require("fs");
+const os = require("os");
+const command_1 = require("../command");
+const logger_1 = require("../logger");
+const projectUtils_1 = require("../projectUtils");
+const requirePermissions_1 = require("../requirePermissions");
+const accountExporter_1 = require("../accountExporter");
+const MAX_BATCH_SIZE = 1000;
+exports.default = new command_1.Command("auth:export [dataFile]")
     .description("Export accounts from your Firebase project into a data file")
-    .option("--format <format>", "Format of exported data (csv, json). Ignored if [dataFile] has format extension.")
-    .before(requirePermissions, ["firebaseauth.users.get"])
-    .action(function (dataFile, options) {
-    var projectId = needProjectId(options);
-    var checkRes = validateOptions(options, dataFile);
+    .option("--format <format>", "Format of exported data (csv, json). Ignored if <dataFile> has format extension.")
+    .before(requirePermissions_1.requirePermissions, ["firebaseauth.users.get"])
+    .action((dataFile, options) => {
+    const projectId = (0, projectUtils_1.needProjectId)(options);
+    const checkRes = (0, accountExporter_1.validateOptions)(options, dataFile);
     if (!checkRes.format) {
         return checkRes;
     }
-    var exportOptions = checkRes;
-    var writeStream = fs.createWriteStream(dataFile);
-    if (exportOptions.format === "json") {
+    const writeStream = fs.createWriteStream(dataFile);
+    if (checkRes.format === "json") {
         writeStream.write('{"users": [' + os.EOL);
     }
-    exportOptions.writeStream = writeStream;
-    exportOptions.batchSize = MAX_BATCH_SIZE;
-    logger.info("Exporting accounts to " + clc.bold(dataFile));
-    return serialExportUsers(projectId, exportOptions).then(function () {
+    const exportOptions = {
+        format: checkRes.format,
+        writeStream,
+        batchSize: MAX_BATCH_SIZE,
+    };
+    logger_1.logger.info("Exporting accounts to " + clc.bold(dataFile));
+    return (0, accountExporter_1.serialExportUsers)(projectId, exportOptions).then(() => {
         if (exportOptions.format === "json") {
             writeStream.write("]}");
         }
         writeStream.end();
-        return new Promise(function (resolve, reject) {
+        return new Promise((resolve, reject) => {
             writeStream.on("finish", resolve);
             writeStream.on("close", resolve);
             writeStream.on("error", reject);

package/lib/commands/auth-import.js

@@ -1,21 +1,23 @@
 "use strict";
-var csv = require("csv-streamify");
-var clc = require("cli-color");
-var fs = require("fs");
-var jsonStream = require("JSONStream");
-var _ = require("lodash");
-var { Command } = require("../command");
-var accountImporter = require("../accountImporter");
-var needProjectId = require("../projectUtils").needProjectId;
-const { logger } = require("../logger");
-var { requirePermissions } = require("../requirePermissions");
-var utils = require("../utils");
-var MAX_BATCH_SIZE = 1000;
-var validateOptions = accountImporter.validateOptions;
-var validateUserJson = accountImporter.validateUserJson;
-var transArrayToUser = accountImporter.transArrayToUser;
-var serialImportUsers = accountImporter.serialImportUsers;
-module.exports = new Command("auth:import [dataFile]")
+Object.defineProperty(exports, "__esModule", { value: true });
+const csv_parse_1 = require("csv-parse");
+const Chain = require("stream-chain");
+const clc = require("cli-color");
+const fs = require("fs-extra");
+const Pick = require("stream-json/filters/Pick");
+const StreamArray = require("stream-json/streamers/StreamArray");
+const command_1 = require("../command");
+const error_1 = require("../error");
+const logger_1 = require("../logger");
+const projectUtils_1 = require("../projectUtils");
+const requirePermissions_1 = require("../requirePermissions");
+const accountImporter = require("../accountImporter");
+const MAX_BATCH_SIZE = 1000;
+const validateOptions = accountImporter.validateOptions;
+const validateUserJson = accountImporter.validateUserJson;
+const transArrayToUser = accountImporter.transArrayToUser;
+const serialImportUsers = accountImporter.serialImportUsers;
+module.exports = new command_1.Command("auth:import [dataFile]")
     .description("import users into your Firebase project from a data file(.csv or .json)")
     .option("--hash-algo <hashAlgo>", "specify the hash algorithm used in password for these accounts")
     .option("--hash-key <hashKey>", "specify the key used in hash algorithm")
@@ -27,81 +29,89 @@ module.exports = new Command("auth:import [dataFile]")
     .option("--dk-len <dkLen>", "specify derived key length for standard scrypt.")
     .option("--hash-input-order <hashInputOrder>", "specify the order of password and salt. Possible values are SALT_FIRST and PASSWORD_FIRST. " +
     "MD5, SHA1, SHA256, SHA512, HMAC_MD5, HMAC_SHA1, HMAC_SHA256, HMAC_SHA512 support this flag.")
-    .before(requirePermissions, ["firebaseauth.users.create", "firebaseauth.users.update"])
-    .action(function (dataFile, options) {
-    var projectId = needProjectId(options);
-    var checkRes = validateOptions(options);
+    .before(requirePermissions_1.requirePermissions, ["firebaseauth.users.create", "firebaseauth.users.update"])
+    .action(async (dataFile, options) => {
+    const projectId = (0, projectUtils_1.needProjectId)(options);
+    const checkRes = validateOptions(options);
     if (!checkRes.valid) {
         return checkRes;
     }
-    var hashOptions = checkRes;
-    if (!_.endsWith(dataFile, ".csv") && !_.endsWith(dataFile, ".json")) {
-        return utils.reject("Data file must end with .csv or .json", { exit: 1 });
+    const hashOptions = checkRes;
+    if (!dataFile.endsWith(".csv") && !dataFile.endsWith(".json")) {
+        throw new error_1.FirebaseError("Data file must end with .csv or .json");
     }
-    var stats = fs.statSync(dataFile);
-    var fileSizeInBytes = stats.size;
-    logger.info("Processing " + clc.bold(dataFile) + " (" + fileSizeInBytes + " bytes)");
-    var inStream = fs.createReadStream(dataFile);
-    var batches = [];
-    var currentBatch = [];
-    var counter = 0;
-    return new Promise(function (resolve, reject) {
-        var parser;
-        if (dataFile.endsWith(".csv")) {
-            parser = csv({ objectMode: true });
+    const stats = await fs.stat(dataFile);
+    const fileSizeInBytes = stats.size;
+    logger_1.logger.info(`Processing ${clc.bold(dataFile)} (${fileSizeInBytes} bytes)`);
+    const batches = [];
+    let currentBatch = [];
+    let counter = 0;
+    let userListArr = [];
+    const inStream = fs.createReadStream(dataFile);
+    if (dataFile.endsWith(".csv")) {
+        userListArr = await new Promise((resolve, reject) => {
+            const parser = (0, csv_parse_1.parse)();
             parser
-                .on("data", function (line) {
-                counter++;
-                var user = transArrayToUser(line.map(function (str) {
-                    var newStr = str.trim().replace(/^["|'](.*)["|']$/, "$1");
-                    return newStr === "" ? undefined : newStr;
-                }));
-                if (user.error) {
-                    return reject("Line " + counter + " (" + line + ") has invalid data format: " + user.error);
-                }
-                currentBatch.push(user);
-                if (currentBatch.length === MAX_BATCH_SIZE) {
-                    batches.push(currentBatch);
-                    currentBatch = [];
+                .on("readable", () => {
+                let record = [];
+                while ((record = parser.read()) !== null) {
+                    counter++;
+                    const trimmed = record.map((s) => {
+                        const str = s.trim().replace(/^["|'](.*)["|']$/, "$1");
+                        return str === "" ? undefined : str;
+                    });
+                    const user = transArrayToUser(trimmed);
+                    const err = user.error;
+                    if (err) {
+                        return reject(new error_1.FirebaseError(`Line ${counter} (${record.join(",")}) has invalid data format: ${err}`));
+                    }
+                    currentBatch.push(user);
+                    if (currentBatch.length === MAX_BATCH_SIZE) {
+                        batches.push(currentBatch);
+                        currentBatch = [];
+                    }
                 }
             })
-                .on("end", function () {
+                .on("end", () => {
                 if (currentBatch.length) {
                     batches.push(currentBatch);
                 }
-                return resolve(batches);
+                resolve(batches);
             });
             inStream.pipe(parser);
-        }
-        else {
-            parser = jsonStream.parse(["users", { emitKey: true }]);
-            parser
-                .on("data", function (pair) {
-                counter++;
-                var res = validateUserJson(pair.value);
-                if (res.error) {
-                    return reject(res.error);
-                }
-                currentBatch.push(pair.value);
-                if (currentBatch.length === MAX_BATCH_SIZE) {
-                    batches.push(currentBatch);
-                    currentBatch = [];
-                }
-            })
-                .on("end", function () {
+        });
+    }
+    else {
+        userListArr = await new Promise((resolve, reject) => {
+            const pipeline = new Chain([
+                Pick.withParser({ filter: /^users$/ }),
+                StreamArray.streamArray(),
+                ({ value }) => {
+                    counter++;
+                    const user = validateUserJson(value);
+                    const err = user.error;
+                    if (err) {
+                        throw new error_1.FirebaseError(`Validation Error: ${err}`);
+                    }
+                    currentBatch.push(user);
+                    if (currentBatch.length === MAX_BATCH_SIZE) {
+                        batches.push(currentBatch);
+                        currentBatch = [];
+                    }
+                },
+            ]);
+            pipeline.once("error", reject);
+            pipeline.on("finish", () => {
                 if (currentBatch.length) {
                     batches.push(currentBatch);
                 }
-                return resolve(batches);
+                resolve(batches);
             });
-            inStream.pipe(parser);
-        }
-    }).then(function (userListArr) {
-        logger.debug("Preparing to import", counter, "user records in", userListArr.length, "batches.");
-        if (userListArr.length) {
-            return serialImportUsers(projectId, hashOptions, userListArr, 0);
-        }
-    }, function (error) {
-        return utils.reject(error, { exit: 1 });
-    });
+            inStream.pipe(pipeline);
+        });
+    }
+    logger_1.logger.debug(`Preparing to import ${counter} user records in ${userListArr.length} batches.`);
+    if (userListArr.length) {
+        return serialImportUsers(projectId, hashOptions, userListArr, 0);
+    }
 });

package/lib/commands/ext-configure.js

@@ -2,7 +2,7 @@
 Object.defineProperty(exports, "__esModule", { value: true });
 const _ = require("lodash");
 const clc = require("cli-color");
-const marked = require("marked");
+const { marked } = require("marked");
 const ora = require("ora");
 const TerminalRenderer = require("marked-terminal");
 const checkMinRequiredVersion_1 = require("../checkMinRequiredVersion");

package/lib/commands/ext-dev-init.js

@@ -2,7 +2,7 @@
 Object.defineProperty(exports, "__esModule", { value: true });
 const fs = require("fs");
 const path = require("path");
-const marked = require("marked");
+const { marked } = require("marked");
 const TerminalRenderer = require("marked-terminal");
 const checkMinRequiredVersion_1 = require("../checkMinRequiredVersion");
 const command_1 = require("../command");

package/lib/commands/ext-dev-publish.js

@@ -1,7 +1,7 @@
 "use strict";
 Object.defineProperty(exports, "__esModule", { value: true });
 const clc = require("cli-color");
-const marked = require("marked");
+const { marked } = require("marked");
 const TerminalRenderer = require("marked-terminal");
 const command_1 = require("../command");
 const extensionsHelper_1 = require("../extensions/extensionsHelper");

package/lib/commands/ext-dev-register.js

@@ -1,7 +1,7 @@
 "use strict";
 Object.defineProperty(exports, "__esModule", { value: true });
 const clc = require("cli-color");
-const marked = require("marked");
+const { marked } = require("marked");
 const command_1 = require("../command");
 const extensionsApi_1 = require("../extensions/extensionsApi");
 const projectUtils_1 = require("../projectUtils");

package/lib/commands/ext-info.js

@@ -10,7 +10,7 @@ const localHelper_1 = require("../extensions/localHelper");
 const logger_1 = require("../logger");
 const requirePermissions_1 = require("../requirePermissions");
 const utils = require("../utils");
-const marked = require("marked");
+const { marked } = require("marked");
 const TerminalRenderer = require("marked-terminal");
 const FUNCTION_TYPE_REGEX = /\..+\.function/;
 exports.default = new command_1.Command("ext:info <extensionName>")

package/lib/commands/ext-install.js

@@ -1,7 +1,7 @@
 "use strict";
 Object.defineProperty(exports, "__esModule", { value: true });
 const clc = require("cli-color");
-const marked = require("marked");
+const { marked } = require("marked");
 const ora = require("ora");
 const TerminalRenderer = require("marked-terminal");
 const askUserForConsent = require("../extensions/askUserForConsent");

package/lib/commands/ext-uninstall.js

@@ -3,7 +3,7 @@ Object.defineProperty(exports, "__esModule", { value: true });
 const _ = require("lodash");
 const clc = require("cli-color");
 const ora = require("ora");
-const marked = require("marked");
+const { marked } = require("marked");
 const TerminalRenderer = require("marked-terminal");
 const checkMinRequiredVersion_1 = require("../checkMinRequiredVersion");
 const command_1 = require("../command");

package/lib/commands/ext-update.js

@@ -2,7 +2,7 @@
 Object.defineProperty(exports, "__esModule", { value: true });
 const clc = require("cli-color");
 const _ = require("lodash");
-const marked = require("marked");
+const { marked } = require("marked");
 const ora = require("ora");
 const TerminalRenderer = require("marked-terminal");
 const checkMinRequiredVersion_1 = require("../checkMinRequiredVersion");

package/lib/commands/functions-config-clone.js

@@ -1,18 +1,20 @@
 "use strict";
-var clc = require("cli-color");
-var { Command } = require("../command");
-var functionsConfig = require("../functionsConfig");
-var functionsConfigClone = require("../functionsConfigClone");
-var needProjectId = require("../projectUtils").needProjectId;
-var { requirePermissions } = require("../requirePermissions");
-var utils = require("../utils");
-const { logger } = require("../logger");
-module.exports = new Command("functions:config:clone")
+Object.defineProperty(exports, "__esModule", { value: true });
+const clc = require("cli-color");
+const command_1 = require("../command");
+const error_1 = require("../error");
+const logger_1 = require("../logger");
+const projectUtils_1 = require("../projectUtils");
+const requirePermissions_1 = require("../requirePermissions");
+const functionsConfig = require("../functionsConfig");
+const functionsConfigClone = require("../functionsConfigClone");
+const utils = require("../utils");
+exports.default = new command_1.Command("functions:config:clone")
     .description("clone environment config from another project")
     .option("--from <projectId>", "the project from which to clone configuration")
     .option("--only <keys>", "a comma-separated list of keys to clone")
     .option("--except <keys>", "a comma-separated list of keys to not clone")
-    .before(requirePermissions, [
+    .before(requirePermissions_1.requirePermissions, [
     "runtimeconfig.configs.list",
     "runtimeconfig.configs.create",
     "runtimeconfig.configs.get",
@@ -25,29 +27,26 @@ module.exports = new Command("functions:config:clone")
     "runtimeconfig.variables.delete",
 ])
     .before(functionsConfig.ensureApi)
-    .action(function (options) {
-    var projectId = needProjectId(options);
+    .action(async (options) => {
+    const projectId = (0, projectUtils_1.needProjectId)(options);
     if (!options.from) {
-        return utils.reject("Must specify a source project in " + clc.bold("--from <projectId>") + " option.");
+        throw new error_1.FirebaseError(`Must specify a source project in ${clc.bold("--from <projectId>")} option.`);
     }
     else if (options.from === projectId) {
-        return utils.reject("From project and destination can't be the same project.");
+        throw new error_1.FirebaseError("From project and destination can't be the same project.");
     }
     else if (options.only && options.except) {
-        return utils.reject("Cannot use both --only and --except at the same time.");
+        throw new error_1.FirebaseError("Cannot use both --only and --except at the same time.");
     }
-    var only;
-    var except;
+    let only = [];
+    let except = [];
     if (options.only) {
         only = options.only.split(",");
     }
     else if (options.except) {
         except = options.except.split(",");
     }
-    return functionsConfigClone(options.from, projectId, only, except).then(function () {
-        utils.logSuccess("Cloned functions config from " + clc.bold(options.from) + " into " + clc.bold(projectId));
-        logger.info("\nPlease deploy your functions for the change to take effect by running " +
-            clc.bold("firebase deploy --only functions") +
-            "\n");
-    });
+    await functionsConfigClone(options.from, projectId, only, except);
+    utils.logSuccess(`Cloned functions config from ${clc.bold(options.from)} into ${clc.bold(projectId)}`);
+    logger_1.logger.info(`\nPlease deploy your functions for the change to take effect by running ${clc.bold("firebase deploy --only functions")}\n`);
 });

package/lib/commands/functions-config-get.js

@@ -1,34 +1,34 @@
 "use strict";
-var _ = require("lodash");
-var { Command } = require("../command");
-var needProjectId = require("../projectUtils").needProjectId;
-const { logger } = require("../logger");
-var { requirePermissions } = require("../requirePermissions");
-var functionsConfig = require("../functionsConfig");
-function _materialize(projectId, path) {
-    if (_.isUndefined(path)) {
+Object.defineProperty(exports, "__esModule", { value: true });
+const lodash_1 = require("lodash");
+const path_1 = require("path");
+const command_1 = require("../command");
+const logger_1 = require("../logger");
+const projectUtils_1 = require("../projectUtils");
+const requirePermissions_1 = require("../requirePermissions");
+const functionsConfig = require("../functionsConfig");
+async function materialize(projectId, path) {
+    if (path === undefined) {
         return functionsConfig.materializeAll(projectId);
     }
-    var parts = path.split(".");
-    var configId = parts[0];
-    var configName = _.join(["projects", projectId, "configs", configId], "/");
-    return functionsConfig.materializeConfig(configName, {}).then(function (result) {
-        var query = _.chain(parts).join(".").value();
-        return query ? _.get(result, query) : result;
-    });
+    const parts = path.split(".");
+    const configId = parts[0];
+    const configName = (0, path_1.join)("projects", projectId, "configs", configId);
+    const result = await functionsConfig.materializeConfig(configName, {});
+    const query = parts.join(".");
+    return query ? (0, lodash_1.get)(result, query) : result;
 }
-module.exports = new Command("functions:config:get [path]")
+exports.default = new command_1.Command("functions:config:get [path]")
     .description("fetch environment config stored at the given path")
-    .before(requirePermissions, [
+    .before(requirePermissions_1.requirePermissions, [
     "runtimeconfig.configs.list",
     "runtimeconfig.configs.get",
     "runtimeconfig.variables.list",
     "runtimeconfig.variables.get",
 ])
     .before(functionsConfig.ensureApi)
-    .action(function (path, options) {
-    return _materialize(needProjectId(options), path).then(function (result) {
-        logger.info(JSON.stringify(result, null, 2));
-        return result;
-    });
+    .action(async (path, options) => {
+    const result = await materialize((0, projectUtils_1.needProjectId)(options), path);
+    logger_1.logger.info(JSON.stringify(result, null, 2));
+    return result;
 });

package/lib/commands/functions-config-set.js

@@ -1,14 +1,16 @@
 "use strict";
-var clc = require("cli-color");
-var { Command } = require("../command");
-var needProjectId = require("../projectUtils").needProjectId;
-var { requirePermissions } = require("../requirePermissions");
-const { logger } = require("../logger");
-var utils = require("../utils");
-var functionsConfig = require("../functionsConfig");
-module.exports = new Command("functions:config:set [values...]")
+Object.defineProperty(exports, "__esModule", { value: true });
+const clc = require("cli-color");
+const command_1 = require("../command");
+const error_1 = require("../error");
+const logger_1 = require("../logger");
+const projectUtils_1 = require("../projectUtils");
+const requirePermissions_1 = require("../requirePermissions");
+const functionsConfig = require("../functionsConfig");
+const utils = require("../utils");
+exports.default = new command_1.Command("functions:config:set [values...]")
     .description("set environment config with key=value syntax")
-    .before(requirePermissions, [
+    .before(requirePermissions_1.requirePermissions, [
     "runtimeconfig.configs.list",
     "runtimeconfig.configs.create",
     "runtimeconfig.configs.get",
@@ -21,20 +23,20 @@ module.exports = new Command("functions:config:set [values...]")
     "runtimeconfig.variables.delete",
 ])
     .before(functionsConfig.ensureApi)
-    .action(function (args, options) {
+    .action(async (args, options) => {
     if (!args.length) {
-        return utils.reject("Must supply at least one key/value pair, e.g. " + clc.bold('app.name="My App"'));
+        throw new error_1.FirebaseError(`Must supply at least one key/value pair, e.g. ${clc.bold('app.name="My App"')}`);
     }
-    var projectId = needProjectId(options);
-    var parsed = functionsConfig.parseSetArgs(args);
-    var promises = [];
-    parsed.forEach(function (item) {
+    const projectId = (0, projectUtils_1.needProjectId)(options);
+    const parsed = functionsConfig.parseSetArgs(args);
+    const promises = [];
+    for (const item of parsed) {
+        if (item.val === undefined) {
+            throw new error_1.FirebaseError(`Unexpected undefined value for varId "${item.varId}`, { exit: 2 });
+        }
         promises.push(functionsConfig.setVariablesRecursive(projectId, item.configId, item.varId, item.val));
-    });
-    return Promise.all(promises).then(function () {
-        utils.logSuccess("Functions config updated.");
-        logger.info("\nPlease deploy your functions for the change to take effect by running " +
-            clc.bold("firebase deploy --only functions") +
-            "\n");
-    });
+    }
+    await Promise.all(promises);
+    utils.logSuccess("Functions config updated.");
+    logger_1.logger.info(`\nPlease deploy your functions for the change to take effect by running ${clc.bold("firebase deploy --only functions")}\n`);
 });