@autofleet/shtinker 1.1.6-beta.13 → 1.1.6-beta.15

package/dist/audit-logger.js

@@ -21,12 +21,17 @@ const getAuditContext = () => {
     const auditContext = (_a = currentTrace === null || currentTrace === void 0 ? void 0 : currentTrace.context) === null || _a === void 0 ? void 0 : _a.get(const_1.AUDIT_LOG_CONTEXT_KEY);
     return auditContext;
 };
-const isEmpty = (value) => {
-    return Array.isArray(value) ? value.length === 0 : value == null;
+const isEmpty = (field) => {
+    if (Array.isArray(field)) {
+        return field.length === 0;
+    }
+    if (typeof field === 'object') {
+        return Object.keys(field).length === 0;
+    }
+    return field == null;
 };
 const filterOutEmptyFields = (fields, instance) => fields.filter(field => !isEmpty(instance.previous(field)) || !isEmpty(instance.get(field)));
 const getChangedFields = (instance, options) => {
-    logger_1.default.info('START getChangedFields');
     // When bulk updating in sequelize using the "returning" option, the instance.changed() stops working.
     // It's a known issut with sequelize.
     if (options.returning) {
@@ -51,7 +56,6 @@ class AuditLogger {
             modelType.addHook('afterSave', (instance, options) => __awaiter(this, void 0, void 0, function* () {
                 try {
                     const auditContext = getAuditContext();
-                    this.logger.info('auditContext:', { auditContext });
                     if (auditContext) {
                         const changedProperties = getChangedFields(instance, options);
                         const payload = {
@@ -65,9 +69,6 @@ class AuditLogger {
                         };
                         this.sendAuditLogRows(payload);
                     }
-                    else {
-                        this.logger.info('NO AUDIT CONTEXT: ', { entityId: instance.id });
-                    }
                 }
                 catch (error) {
                     logger_1.default.error('Failed to send audit log rows', error);

package/dist/index.js

@@ -42,7 +42,6 @@ exports.enableAuditing = enableAuditing;
 const setAuditContext = (entityType, action) => (req, res, next) => __awaiter(void 0, void 0, void 0, function* () {
     try {
         const currentTrace = (0, zehut_1.getCurrentPayload)();
-        logger_1.default.info('getCurrentTrace', { getCurrentTrace: zehut_1.getCurrentPayload });
         if (currentTrace && currentTrace.context && currentTrace.context.get) {
             const user = currentTrace.context.get(const_1.USER_CONTEXT_KEY);
             const auditLogContext = {

package/package.json

@@ -1,6 +1,6 @@
 {
   "name": "@autofleet/shtinker",
-  "version": "1.1.6-beta.13",
+  "version": "1.1.6-beta.15",
   "description": "",
   "main": "dist/index.js",
   "scripts": {

package/src/audit-logger.ts

@@ -12,14 +12,19 @@ const getAuditContext = () => {
   return auditContext;
 };
 
-const isEmpty = (value) => {
-  return Array.isArray(value) ? value.length === 0 : value == null;
-};
+const isEmpty = (field) => {
+  if (Array.isArray(field)) {
+    return field.length === 0;
+  }
+  if (typeof field === 'object') {
+    return Object.keys(field).length === 0;
+  }
+  return field == null;
+}
 
-const filterOutEmptyFields = (fields, instance) => fields.filter(field => !isEmpty(instance.previous(field)) || !isEmpty(instance.get(field)))
+const filterOutEmptyFields = (fields, instance) => fields.filter(field => !isEmpty(instance.previous(field)) || !isEmpty(instance.get(field)));
 
 const getChangedFields = (instance, options) => {
-  logger.info('START getChangedFields');
   // When bulk updating in sequelize using the "returning" option, the instance.changed() stops working.
   // It's a known issut with sequelize.
   if (options.returning) {
@@ -52,7 +57,6 @@ class AuditLogger {
       modelType.addHook('afterSave', async (instance: any, options: any) => {
         try {
           const auditContext = getAuditContext();
-          this.logger.info('auditContext:', { auditContext });
           if (auditContext) {
             const changedProperties = getChangedFields(instance, options);
             const payload: AuditLogPayload = {
@@ -65,8 +69,6 @@ class AuditLogger {
               })),
             };
             this.sendAuditLogRows(payload);
-          } else {
-            this.logger.info('NO AUDIT CONTEXT: ', { entityId: instance.id });
           }
         } catch (error) {
           logger.error('Failed to send audit log rows', error);

package/src/index.ts

@@ -19,7 +19,6 @@ export const setAuditContext = (
 ) => async (req: any, res: any, next: any): Promise<any> => {
   try {
     const currentTrace = getCurrentTrace();
-    logger.info('getCurrentTrace', { getCurrentTrace });
     if (currentTrace && currentTrace.context && currentTrace.context.get) {
       const user = currentTrace.context.get(USER_CONTEXT_KEY);
       const auditLogContext: AuditLogContext = {