remove removeTemporaryLogAttributes, refactor removeKeys

Author: shdq

packages/logger/src/Logger.ts

@@ -511,42 +511,31 @@ class Logger extends Utility implements LoggerInterface {
   }
 
   /**
-   * Alias for removePersistentLogAttributes.
+   * It removes attributes based on provided keys to all log items generated by this Logger instance.
    *
    * @param {string[]} keys
    * @returns {void}
    */
   public removeKeys(keys: string[]): void {
-    this.removePersistentLogAttributes(keys);
-    this.removeTemporaryLogAttributes(keys);
-  }
-
-  /**
-   * It removes persistent attributes based on provided keys to all log items generated by this Logger instance.
-   *
-   * @param {string[]} keys
-   * @returns {void}
-   */
-  public removePersistentLogAttributes(keys: string[]): void {
-    for (const key of keys) {
-      if (this.persistentLogAttributes && key in this.persistentLogAttributes) {
-        delete this.persistentLogAttributes[key];
+    const removeKey = (attributes: LogAttributes, key: string): void => {
+      if (attributes && key in attributes) {
+        delete attributes[key];
       }
+    };
+    for (const key of keys) {
+      removeKey(this.persistentLogAttributes, key);
+      removeKey(this.temporaryLogAttributes, key);
     }
   }
 
   /**
-   * It removes temporary attributes based on provided keys to all log items generated by this Logger instance.
+   * Alias for removeKeys().
    *
    * @param {string[]} keys
    * @returns {void}
    */
-  public removeTemporaryLogAttributes(keys: string[]): void {
-    for (const key of keys) {
-      if (this.temporaryLogAttributes && key in this.temporaryLogAttributes) {
-        delete this.temporaryLogAttributes[key];
-      }
-    }
+  public removePersistentLogAttributes(keys: string[]): void {
+    this.removeKeys(keys);
   }
 
   /**