@salesforce/core 3.13.0 → 3.13.1

package/CHANGELOG.md

@@ -2,6 +2,12 @@
 
 All notable changes to this project will be documented in this file. See [standard-version](https://github.com/conventional-changelog/standard-version) for commit guidelines.
 
+### [3.13.1](https://github.com/forcedotcom/sfdx-core/compare/v3.13.0...v3.13.1) (2022-04-21)
+
+### Bug Fixes
+
+- log rotation errors ([#564](https://github.com/forcedotcom/sfdx-core/issues/564)) ([8a4c9de](https://github.com/forcedotcom/sfdx-core/commit/8a4c9de85840ae0efa0ee8a27494d63a61545da5))
+
 ## [3.13.0](https://github.com/forcedotcom/sfdx-core/compare/v3.12.2...v3.13.0) (2022-04-20)
 
 ### Features

package/lib/config/config.js

@@ -42,7 +42,6 @@ const messages = messages_1.Messages.load('@salesforce/core', 'config', [
     'instanceUrl',
     'disable-telemetry',
 ]);
-const log = logger_1.Logger.childFromRoot('core:config');
 const SFDX_CONFIG_FILE_NAME = 'sfdx-config.json';
 const CONFIG_FILE_NAME = 'config.json';
 var SfConfigProperties;
@@ -277,9 +276,12 @@ class Config extends configFile_1.ConfigFile {
      */
     static addAllowedProperties(metas) {
         const currentMetaKeys = Object.keys(Config.propertyConfigMap());
+        // If logger is needed elsewhere in this file, do not move this outside of the Config class.
+        // It was causing issues with Bunyan log rotation. See https://github.com/forcedotcom/sfdx-core/pull/562
+        const logger = logger_1.Logger.childFromRoot('core:config');
         metas.forEach((meta) => {
             if (currentMetaKeys.includes(meta.key)) {
-                log.info(`Key ${meta.key} already exists in allowedProperties, skipping.`);
+                logger.info(`Key ${meta.key} already exists in allowedProperties, skipping.`);
                 return;
             }
             Config.allowedProperties.push(meta);

package/lib/globalInfo/sfdxDataHandler.js

@@ -31,10 +31,8 @@ class SfdxDataHandler {
         this.handlers = [new AuthHandler(), new AliasesHandler()];
     }
     async write(latest = globalInfoConfig_1.GlobalInfo.emptyDataModel) {
-        for (const handler of this.handlers) {
-            await handler.write(latest, this.original);
-            this.setOriginal(latest);
-        }
+        await Promise.all(this.handlers.map((handler) => handler.write(latest, this.original)));
+        this.setOriginal(latest);
     }
     async merge(sfData = globalInfoConfig_1.GlobalInfo.emptyDataModel) {
         let merged = (0, globalInfoConfig_1.deepCopy)(sfData);
@@ -103,17 +101,17 @@ class AuthHandler extends BaseHandler {
     }
     async write(latest, original) {
         const { changed, deleted } = await this.findChanges(latest, original);
-        for (const [username, authData] of Object.entries(changed)) {
-            if (authData) {
-                const config = await this.createAuthFileConfig(username);
-                config.setContentsFromObject(authData);
-                await config.write();
-            }
-        }
-        for (const username of deleted) {
+        await Promise.all(Object.entries(changed)
+            .filter(([, authData]) => authData)
+            .map(async ([username, authData]) => {
             const config = await this.createAuthFileConfig(username);
-            await config.unlink();
-        }
+            config.setContentsFromObject(authData);
+            return config.write();
+        }));
+        await Promise.all(deleted.map(async (username) => {
+            const config = await this.createAuthFileConfig(username);
+            return config.unlink();
+        }));
     }
     async findChanges(latest, original) {
         var _a;
@@ -146,16 +144,14 @@ class AuthHandler extends BaseHandler {
     }
     async listAllAuthorizations() {
         const filenames = await this.listAllAuthFiles();
-        const auths = [];
-        for (const filename of filenames) {
-            const username = (0, path_1.basename)(filename, (0, path_1.extname)(filename));
+        return Promise.all(filenames
+            .map((f) => (0, path_1.basename)(f, (0, path_1.extname)(f)))
+            .map(async (username) => {
             const configFile = await this.createAuthFileConfig(username);
             const contents = configFile.getContents();
             const stat = await configFile.stat();
-            const auth = Object.assign(contents, { timestamp: stat.mtime.toISOString() });
-            auths.push(auth);
-        }
-        return auths;
+            return { ...contents, timestamp: stat.mtime.toISOString() };
+        }));
     }
 }
 exports.AuthHandler = AuthHandler;

package/package.json

@@ -1,6 +1,6 @@
 {
   "name": "@salesforce/core",
-  "version": "3.13.0",
+  "version": "3.13.1",
   "description": "Core libraries to interact with SFDX projects, orgs, and APIs.",
   "main": "lib/exported",
   "types": "lib/exported.d.ts",