@cap-js-community/event-queue 1.10.7 → 1.10.9

package/package.json

@@ -1,6 +1,6 @@
 {
   "name": "@cap-js-community/event-queue",
-  "version": "1.10.7",
+  "version": "1.10.9",
   "description": "An event queue that enables secure transactional processing of asynchronous and periodic events, featuring instant event processing with Redis Pub/Sub and load distribution across all application instances.",
   "main": "src/index.js",
   "types": "src/index.d.ts",

package/src/runner/openEvents.js

@@ -38,34 +38,27 @@ const getOpenQueueEntries = async (tx, filterAppSpecificEvents = true) => {
   for (const { type, subType } of entries) {
     if (eventConfig.isCapOutboxEvent(type)) {
       const [srvName, actionName] = subType.split(".");
-      cds.connect
-        .to(srvName)
-        .then((service) => {
-          if (!filterAppSpecificEvents) {
-            return; // will be done in finally
-          }
-
+      try {
+        const service = await cds.connect.to(srvName);
+        if (filterAppSpecificEvents) {
           if (!service) {
-            return;
+            continue;
           }
           cds.outboxed(service);
           if (actionName) {
             config.addCAPOutboxEventSpecificAction(srvName, actionName);
           }
-          if (filterAppSpecificEvents) {
-            if (eventConfig.shouldBeProcessedInThisApplication(type, subType)) {
-              result.push({ type, subType });
-            }
-          } else {
-            result.push({ type, subType });
-          }
-        })
-        .catch(() => {})
-        .finally(() => {
-          if (!filterAppSpecificEvents) {
+          if (eventConfig.shouldBeProcessedInThisApplication(type, subType)) {
             result.push({ type, subType });
           }
-        });
+        }
+      } catch {
+        /* ignore catch */
+      } finally {
+        if (!filterAppSpecificEvents) {
+          result.push({ type, subType });
+        }
+      }
     } else {
       if (filterAppSpecificEvents) {
         if (

package/src/shared/cdsHelper.js

@@ -150,7 +150,9 @@ const getAllTenantIds = async () => {
     }, []);
 };
 
-const getAllTenantWithSubdomain = async () => {
+const TENANT_COLUMNS = ["subscribedSubdomain", "createdAt", "modifiedAt"];
+
+const getAllTenantWithMetadata = async () => {
   const response = await _getAllTenantBase();
   if (!response) {
     return null;
@@ -160,10 +162,19 @@ const getAllTenantWithSubdomain = async () => {
     const tenantId = row.subscribedTenantId ?? row.tenant;
     result = await result;
     if (await common.isTenantIdValidCb(TenantIdCheckTypes.eventProcessing, tenantId)) {
-      result.push({
-        ID: tenantId,
-        subdomain: row.subscribedSubdomain,
-      });
+      const data = Object.entries(row).reduce(
+        (result, [key, value]) => {
+          if (TENANT_COLUMNS.includes(key)) {
+            result[key] = value;
+          } else {
+            result.metadata[key] = value;
+          }
+          return result;
+        },
+        { metadata: {} }
+      );
+      data.metadata = JSON.stringify(data.metadata);
+      result.push(data);
     }
     return result;
   }, []);
@@ -172,5 +183,5 @@ const getAllTenantWithSubdomain = async () => {
 module.exports = {
   executeInNewTransaction,
   getAllTenantIds,
-  getAllTenantWithSubdomain,
+  getAllTenantWithMetadata,
 };

package/src/shared/distributedLock.js

@@ -121,8 +121,9 @@ const _checkLockExistsDb = async (context, fullKey) => {
 
 const _releaseLockRedis = async (context, fullKey) => {
   const client = await redis.createMainClientAndConnect(config.redisOptions);
-  await client.del(fullKey);
+  const result = await client.del(fullKey);
   delete existingLocks[fullKey];
+  return result === 1;
 };
 
 const _releaseLockDb = async (context, fullKey) => {
@@ -130,6 +131,7 @@ const _releaseLockDb = async (context, fullKey) => {
     await tx.run(DELETE.from(config.tableNameEventLock).where("code =", fullKey));
   });
   delete existingLocks[fullKey];
+  return true;
 };
 
 const _acquireLockDB = async (

package/srv/service/admin-service.cds

@@ -13,14 +13,14 @@ service EventQueueAdminService {
         null as space: String,
         *
   } actions {
-    action setStatusAndAttempts(
-      // TODO: remove tenant as soon as CAP issue is fixed https://github.tools.sap/cap/issues/issues/18445
-      @mandatory
-      tenant: String,
-      status: db.Status,
-      @assert.range: [0,100]
-      attempts: Integer) returns Event;
-  }
+      action setStatusAndAttempts(
+        // TODO: remove tenant as soon as CAP issue is fixed https://github.tools.sap/cap/issues/issues/18445
+        @mandatory
+        tenant: String,
+        status: db.Status,
+        @assert.range: [0,100]
+        attempts: Integer) returns Event;
+    }
 
   @cds.persistence.skip
   @readonly
@@ -32,12 +32,22 @@ service EventQueueAdminService {
     space: String;
     ttl: Integer;
     createdAt: Integer;
-  }
+  } actions {
+      action releaseLock(
+        // TODO: remove tenant as soon as CAP issue is fixed https://github.tools.sap/cap/issues/issues/18445
+        @mandatory
+        tenant: String,
+        @mandatory
+        type: String,
+        @mandatory
+        subType: String) returns Boolean;
+    }
 
    @readonly
    @cds.persistence.skip
    entity Tenant {
       Key ID: String;
       subdomain: String;
+      metadata: String;
    }
 }

package/srv/service/admin-service.js

@@ -5,6 +5,7 @@ const cdsHelper = require("../../src/shared/cdsHelper");
 const { EventProcessingStatus } = require("../../src");
 const config = require("../../src/config");
 const distributedLock = require("../../src/shared/distributedLock");
+const redisPub = require("../../src/redis/redisPub");
 
 module.exports = class AdminService extends cds.ApplicationService {
   async init() {
@@ -61,7 +62,7 @@ module.exports = class AdminService extends cds.ApplicationService {
     });
 
     this.on("READ", Tenant, async () => {
-      const tenants = await cdsHelper.getAllTenantWithSubdomain();
+      const tenants = await cdsHelper.getAllTenantWithMetadata();
       return tenants ?? [];
     });
 
@@ -82,14 +83,27 @@ module.exports = class AdminService extends cds.ApplicationService {
         return req.reject(400, "No status or attempts provided");
       }
 
-      await cds.tx({ tenant, headers: { "z-id": tenant } }, async () => {
+      const event = await cds.tx({ tenant, headers: { "z-id": tenant } }, async () => {
+        const event = await SELECT.one.from(EventDb).where({ ID: req.params[0].ID ?? req.params[0] });
         await UPDATE.entity(EventDb)
           .set(updateData)
           .where({ ID: req.params[0].ID ?? req.params[0] });
+        return event;
+      });
+      redisPub.broadcastEvent(tenant, event).catch(() => {
+        /* ignore errors */
       });
       return await this.send(new cds.Request({ query: req.query, headers: req.headers }));
     });
 
+    this.on("releaseLock", async (req) => {
+      cds.log("eventQueue").info("Releasing event-queue lock", req.data);
+      const { tenant, type, subType } = req.data;
+      return await cds.tx({ tenant }, async (tx) => {
+        return await distributedLock.releaseLock(tx.context, [type, subType].join("##"));
+      });
+    });
+
     await super.init();
   }